Skip to content

Overlay Function#16

Open
Xivadas wants to merge 2 commits into
tymwitko:mainfrom
Xivadas:overlay-function
Open

Overlay Function#16
Xivadas wants to merge 2 commits into
tymwitko:mainfrom
Xivadas:overlay-function

Conversation

@Xivadas
Copy link
Copy Markdown

@Xivadas Xivadas commented May 31, 2026

Added overlay function

@Xivadas
Copy link
Copy Markdown
Author

Xivadas commented May 31, 2026

Its working on my end. Let me know if there are any issues. I might've pushed the commit wrong!

@tymwitko tymwitko self-requested a review May 31, 2026 18:02

fun canUseShizuku(): Boolean {
return ShizukuShell.isAvailable()
}
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Root is also available, so it should be considered just as well. If you do it via RootBeer, remember to put it into a coroutine, because it's pretty resource-heavy as I've already learned

.padding(horizontal = 16.dp)
.pointerInput(Unit) {
detectTapGestures {
// Consume taps inside the overlay card so only outside taps close it.
Copy link
Copy Markdown
Owner

@tymwitko tymwitko May 31, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't believe it's necessary, if you remove the whole .pointerInput block, it should consume taps just as well (but I could be wrong here, so double-check)


Text(
text = if (hasPrivileges) {
"Tap to open • Swipe up/down to kill • Scroll left/right"
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extract all strings to strings.xml to facilitate localizing


private const val DEFAULT_REFRESH_SECONDS = 10

fun getRefreshSeconds(context: Context): Int {
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Better to inject SharedPreferences into a manager, just like it's done in SettingsHolder

@tymwitko
Copy link
Copy Markdown
Owner

I see your commit alright, but it appears you made your changes on top of an older commit, reverting many changes in the process :/

Please try to revert all changes this PR introduces to files that aren't relevant to the overlay functionality, as I also see you made changes to e.g. how Shizuku is managed (keep it within ShizukuManager) or how apps are being extracted (was this vibecoded?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants