Skip to content

manager(refactor): change kernel umount to a switch preference, figure the sucompte string problem#3146

Merged
YuKongA merged 5 commits into
tiann:mainfrom
MiRinChan:main
Jan 18, 2026
Merged

manager(refactor): change kernel umount to a switch preference, figure the sucompte string problem#3146
YuKongA merged 5 commits into
tiann:mainfrom
MiRinChan:main

Conversation

@MiRinChan

@MiRinChan MiRinChan commented Jan 10, 2026

Copy link
Copy Markdown
Contributor
  • Update Settings.kt to use SuperSwitch instead of SuperDropdown for the kernel umount setting.
  • Change the string of sucompte to prevent misunderstanding.

Known issue:
By long string, line brake looks broken on options. This issue is may caused by miuix.

@MiRinChan MiRinChan changed the title manager(refactor): change kernel umount to a switch preference, figure the sucompte string prroblem manager(refactor): change kernel umount to a switch preference, figure the sucompte string problem Jan 10, 2026
@YuKongA YuKongA requested a review from Copilot January 11, 2026 08:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This pull request refactors the kernel umount setting from a dropdown (SuperDropdown) to a switch (SuperSwitch) for better user experience, and updates string resources to improve clarity by renaming "sucompte" to "sucompat" and revising related text across all language files.

Changes:

  • Changed kernel umount UI from SuperDropdown with three modes to SuperSwitch with simple on/off toggle
  • Renamed string resources from "settings_disable_su" to "settings_sucompat" and "settings_disable_kernel_umount" to "settings_kernel_umount" for better clarity
  • Added new string resources for mode options and updated translations across 40+ language files

Reviewed changes

Copilot reviewed 45 out of 45 changed files in this pull request and generated 7 comments.

File Description
manager/app/src/main/java/me/weishu/kernelsu/ui/screen/Settings.kt Refactored kernel umount from dropdown to switch, simplified state management, updated string resource references
manager/app/src/main/res/values/strings.xml Updated English string resources with clearer naming (sucompat instead of disable_su, kernel_umount instead of disable_kernel_umount)
manager/app/src/main/res/values-*/strings.xml Applied consistent string updates across all 40+ localization files

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread manager/app/src/main/res/values/strings.xml Outdated
Comment thread manager/app/src/main/res/values/strings.xml
Comment thread manager/app/src/main/res/values/strings.xml
Comment thread manager/app/src/main/res/values-ja/strings.xml Outdated
Comment thread manager/app/src/main/java/me/weishu/kernelsu/ui/screen/Settings.kt Outdated
Comment thread manager/app/src/main/java/me/weishu/kernelsu/ui/screen/Settings.kt Outdated
Comment thread manager/app/src/main/java/me/weishu/kernelsu/ui/screen/Settings.kt Outdated
@aviraxp

aviraxp commented Jan 11, 2026

Copy link
Copy Markdown
Collaborator

应该是:允许应用通过执行 /system/bin/su 获取超级用户 shell

Comment thread manager/app/src/main/res/values-zh-rCN/strings.xml Outdated
Comment thread manager/app/src/main/java/me/weishu/kernelsu/ui/screen/Settings.kt
@MiRinChan MiRinChan marked this pull request as draft January 16, 2026 13:33
@MiRinChan MiRinChan force-pushed the main branch 2 times, most recently from e1032c9 to d7cd85f Compare January 16, 2026 14:15
…e the sucompat string problem

Co-authored-by: Weblate (bot) <hosted@weblate.org>
Co-authored-by: yuztass <inkognito0901@gmail.com>
Co-authored-by: Beni Fachrudin <neebexd@gmail.com>
Co-authored-by: dabao1955 <dabao1955@163.com>
Co-authored-by: Léane GRASSER <leane.grasser@proton.me>
Co-authored-by: SchweGELBin <abramjannikmichael06@gmail.com>
Co-authored-by: searinminecraft <kitakita@disroot.org>
@MiRinChan MiRinChan marked this pull request as ready for review January 16, 2026 15:15
@MiRinChan MiRinChan marked this pull request as draft January 16, 2026 15:35
@MiRinChan MiRinChan marked this pull request as ready for review January 16, 2026 15:51
Comment thread manager/app/src/main/res/values-my/strings.xml Outdated
@YuKongA YuKongA merged commit cc59210 into tiann:main Jan 18, 2026
KOWX712 pushed a commit to KOWX712/KernelSU that referenced this pull request Jan 18, 2026
…e the sucompte string problem (tiann#3146)

- Update `Settings.kt` to use `SuperSwitch` instead of `SuperDropdown`
for the kernel umount setting.
- Change the string of sucompte to prevent misunderstanding.

---------

Co-authored-by: Weblate (bot) <hosted@weblate.org>
Co-authored-by: yuztass <inkognito0901@gmail.com>
Co-authored-by: Beni Fachrudin <neebexd@gmail.com>
Co-authored-by: dabao1955 <dabao1955@163.com>
Co-authored-by: Léane GRASSER <leane.grasser@proton.me>
Co-authored-by: SchweGELBin <abramjannikmichael06@gmail.com>
Co-authored-by: searinminecraft <kitakita@disroot.org>
Co-authored-by: YuKongA <70465933+YuKongA@users.noreply.github.com>
u9521 added a commit to u9521/KernelSU that referenced this pull request Jan 20, 2026
…e the sucompte string problem (tiann#3146)

- Update `Settings.kt` to use `SuperSwitch` instead of `SuperDropdown`
for the kernel umount setting.
- Change the string of sucompte to prevent misunderstanding.

---------

Co-authored-by: Weblate (bot) <hosted@weblate.org>
Co-authored-by: yuztass <inkognito0901@gmail.com>
Co-authored-by: Beni Fachrudin <neebexd@gmail.com>
Co-authored-by: dabao1955 <dabao1955@163.com>
Co-authored-by: Léane GRASSER <leane.grasser@proton.me>
Co-authored-by: SchweGELBin <abramjannikmichael06@gmail.com>
Co-authored-by: searinminecraft <kitakita@disroot.org>
Co-authored-by: YuKongA <70465933+YuKongA@users.noreply.github.com>
Co-authored-by: 米凛MiRin <148533509+mirinchan@users.noreply.github.com>
Signed-off-by: u9521 <63995396+u9521@users.noreply.github.com>
u9521 added a commit to u9521/KernelSU that referenced this pull request Jan 20, 2026
…e the sucompte string problem (tiann#3146)

- Update `Settings.kt` to use `SuperSwitch` instead of `SuperDropdown`
for the kernel umount setting.
- Change the string of sucompte to prevent misunderstanding.

---------

Co-authored-by: Weblate (bot) <hosted@weblate.org>
Co-authored-by: yuztass <inkognito0901@gmail.com>
Co-authored-by: Beni Fachrudin <neebexd@gmail.com>
Co-authored-by: dabao1955 <dabao1955@163.com>
Co-authored-by: Léane GRASSER <leane.grasser@proton.me>
Co-authored-by: SchweGELBin <abramjannikmichael06@gmail.com>
Co-authored-by: searinminecraft <kitakita@disroot.org>
Co-authored-by: YuKongA <70465933+YuKongA@users.noreply.github.com>
Co-authored-by: 米凛MiRin <148533509+mirinchan@users.noreply.github.com>
Signed-off-by: u9521 <63995396+u9521@users.noreply.github.com>
u9521 added a commit to u9521/KernelSU that referenced this pull request Jan 20, 2026
…e the sucompte string problem (tiann#3146)

- Update `Settings.kt` to use `SuperSwitch` instead of `SuperDropdown`
for the kernel umount setting.
- Change the string of sucompte to prevent misunderstanding.

---------

Co-authored-by: Weblate (bot) <hosted@weblate.org>
Co-authored-by: yuztass <inkognito0901@gmail.com>
Co-authored-by: Beni Fachrudin <neebexd@gmail.com>
Co-authored-by: dabao1955 <dabao1955@163.com>
Co-authored-by: Léane GRASSER <leane.grasser@proton.me>
Co-authored-by: SchweGELBin <abramjannikmichael06@gmail.com>
Co-authored-by: searinminecraft <kitakita@disroot.org>
Co-authored-by: YuKongA <70465933+YuKongA@users.noreply.github.com>
Co-authored-by: 米凛MiRin <148533509+mirinchan@users.noreply.github.com>
Signed-off-by: u9521 <63995396+u9521@users.noreply.github.com>
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.

5 participants