Skip to content

manager: Adjust the order of the theme settings items#3273

Closed
WenHao2130 wants to merge 1 commit into
tiann:mainfrom
WenHao2130:wenhao-patch-1
Closed

manager: Adjust the order of the theme settings items#3273
WenHao2130 wants to merge 1 commit into
tiann:mainfrom
WenHao2130:wenhao-patch-1

Conversation

@WenHao2130

Copy link
Copy Markdown
Contributor

This commit swaps the menu order of "Theme" and "UI Style" to match the menu order in Miuix.

Change-Id: I9e89815046d79e16e351fc09128856ad1a775e62

This commit swaps the menu order of "Theme" and "UI Style" to match the menu order in Miuix.

Change-Id: I9e89815046d79e16e351fc09128856ad1a775e62
Signed-off-by: WenHao2130 <wenhao2130@outlook.com>

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 PR reorders the theme settings items in the Material UI settings screen so that "UI Style" (UI Mode dropdown) appears before "Theme" (color palette navigation), matching the existing order in the Miuix settings screen.

Changes:

  • Moved the SegmentedDropdownItem for UI Mode above the SegmentedListItem for Theme in SettingPagerMaterial.

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

@KOWX712

KOWX712 commented Mar 12, 2026

Copy link
Copy Markdown
Collaborator

more to-be-changed here https://github.com/KOWX712/KernelSU/commits/ksu_ui/

Ylarod pushed a commit that referenced this pull request Mar 12, 2026
- add missing markNeedRefresh in superuser screen
- sync setting/colorpalette icon with miuix (duplicate with #3273)
- miragte module repo detail page to LargeFlexibleTopAppBar
- clean up unused strings
@Ylarod Ylarod closed this Mar 12, 2026
KOWX712 added a commit to KOWX712/KernelSU that referenced this pull request Mar 12, 2026
- add missing markNeedRefresh in superuser screen
- sync setting/colorpalette icon with miuix (duplicate with tiann#3273)
- miragte module repo detail page to LargeFlexibleTopAppBar
- clean up unused strings
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.

4 participants