Skip to content

Conversation

@binhmod
Copy link

@binhmod binhmod commented Jan 2, 2026

Previously, the loading state was shared across all start options
(Root, Shizuku, ADB), causing multiple start buttons to display a
loading indicator simultaneously.

This PR scopes the loading state to the selected start method by
checking startingMethod, ensuring that only the active start option
shows a loading indicator. The loading state is also cleared correctly
once the system bridge connects.

Translated from Vietnamese to English using Google Translate. Thank you.

@juangarcia15894-a11y
Copy link

Buena

@sds100 sds100 added this to the 4.0 milestone Jan 4, 2026
@sds100
Copy link
Collaborator

sds100 commented Jan 4, 2026

All buttons need to show the loading indicator (or at least be disabled) because you shouldn't be able to start expert mode with root while you're starting it with shizuku for example.

What do you think? Thanks for contributing though :)

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.

3 participants