Skip to content

Correcting color to match rosepine moon + sidebar text actually white#1263

Open
yoy333 wants to merge 3 commits into
spicetify:masterfrom
yoy333:master
Open

Correcting color to match rosepine moon + sidebar text actually white#1263
yoy333 wants to merge 3 commits into
spicetify:masterfrom
yoy333:master

Conversation

@yoy333
Copy link
Copy Markdown

@yoy333 yoy333 commented Feb 18, 2026

https://rosepinetheme.com/palette/ingredients/

As you can see here, the moon variant was incorrect. Also the sidebar text was rose, which imo is not good looking.

image

Summary by CodeRabbit

  • Style
    • Updated the rose-pine-moon color palette: refined colors for selected rows, buttons (including active state), and active tabs to improve visual consistency and overall UI appearance.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 18, 2026

No actionable comments were generated in the recent review. 🎉


Walkthrough

The rose-pine-moon palette in Ziro/color.ini had four hex color values changed: selected-row, button, button-active, and tab-active.

Changes

Cohort / File(s) Summary
Color Scheme Configuration
Ziro/color.ini
Updated four hex color values in the rose-pine-moon palette: selected-row, button, button-active, and tab-active (four replacements, no other keys changed).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • kyrie25

Poem

🐰✨ A nibble of color, a tiny tune,
Rose-pine-moon shifts beneath the moon,
Four little hues hopped into place,
Buttons and tabs now wear new grace,
I jiggle my whiskers — palette, hooray!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main changes: correcting rose-pine-moon color values and updating sidebar text color to white.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@Ziro/color.ini`:
- Around line 239-240: The color values for the keys selected-row and button
include trailing spaces ("e0def4 " and "ea9a97 "); remove the trailing
whitespace so the values are exactly the 6-character hex strings (e.g., "e0def4"
and "ea9a97") in the Ziro/color.ini file to avoid producing 7-character invalid
hex values—update the entries for selected-row and button accordingly and ensure
consistency with the other color lines.

Comment thread Ziro/color.ini Outdated
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@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.

1 participant