Skip to content

fix(webui): The internal/colors.css is empty source when WebUIActivity started by shortcut#3172

Merged
YuKongA merged 1 commit into
tiann:mainfrom
lamprose:webui-shortcut-monet-fix
Jan 21, 2026
Merged

fix(webui): The internal/colors.css is empty source when WebUIActivity started by shortcut#3172
YuKongA merged 1 commit into
tiann:mainfrom
lamprose:webui-shortcut-monet-fix

Conversation

@lamprose

Copy link
Copy Markdown
Contributor

Applies the KernelSUTheme to the WebUIActivity, ensuring it provide not empty css when it started by shortcut

…ity started by shortcut

Applies the `KernelSUTheme` to the `WebUIActivity`, ensuring it provide not empty css when it started by shortcut
@YuKongA YuKongA requested a review from Copilot January 21, 2026 08:27

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 fixes an issue where the internal/colors.css file was empty when WebUIActivity was started by a shortcut. The root cause was that the KernelSUTheme composable was not being applied, which meant the UpdateCss() function inside the theme was never called to populate the CSS colors for Monet themes.

Changes:

  • Wrapped the loading indicator in KernelSUTheme to ensure theme colors are properly initialized
  • Added necessary Compose imports for state management and theme support
  • Applied the same theme initialization pattern used in MainActivity to WebUIActivity

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

@YuKongA YuKongA merged commit 93bbf4e into tiann:main Jan 21, 2026
6 checks passed
KOWX712 pushed a commit to KOWX712/KernelSU that referenced this pull request Jan 25, 2026
…ity started by shortcut (tiann#3172)

Applies the `KernelSUTheme` to the `WebUIActivity`, ensuring it provide
not empty css when it started by shortcut
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