🤖 fix: adjust solarized themes for sidebar icons and accent colors #898
+99
−51
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add visual separation in solarized-dark: workspace darker than sidebar - Add --color-runtime-* CSS variables using solarized palette: - SSH: blue (#268bd2) - Worktree: violet (#6c71c4) - Local: base0 (#839496) - Update thinking colors to use solarized violet - Refactor RuntimeBadge and RuntimeIconSelector to use CSS vars instead of hardcoded Tailwind colors for theme adaptability _Generated with mux_
Replace hardcoded HSL values with --color-thinking-mode CSS variable so the slider adapts to theme (especially solarized). _Generated with mux_
The color-mix syntax wasn't working in inline styles. Use the CSS variables directly: --color-thinking-mode-light for LOW, --color-thinking-mode for MEDIUM/HIGH. _Generated with mux_
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
Reverts the runtime icon colors in default light/dark themes to match the original Tailwind values (blue-500, purple-500) exactly, avoiding visual regression. Solarized themes keep the new solarized-palette values. _Generated with mux_
- Restore runtime icon text colors in default themes to match original (lighter shades) - Use --color-text-secondary for ThinkingSlider 'off' state to match original gray - Ensure Solarized themes use consistent palette colors _Generated with mux_
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Adjusts the solarized themes to properly use the solarized color palette for:
Changes
Theme colors
--color-runtime-*CSS variables using solarized palette:#268bd2)#6c71c4)#839496)#6c71c4)Component updates
RuntimeBadgeandRuntimeIconSelectorto use CSS variables instead of hardcoded Tailwind colorsThinkingSliderto use--color-thinking-modeCSS variable for label and glow effectsGenerated with
mux