-
Notifications
You must be signed in to change notification settings - Fork 219
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix terminal selection colors when transparency is turned on #1584
Conversation
…was too washed out and not using the correct official colors)
WalkthroughThe pull request introduces modifications to the In the "onedarkpro" theme, several color values are updated, including background, red, green, blue, and magenta colors. These modifications slightly alter the theme's visual appearance by tweaking specific color hex codes. The "dracula", "monokai", and "rosepine" themes see the removal of the The "campbell" and "warmyellow" themes experience modifications to their These changes represent fine-tuning of the color configurations, potentially to improve visual consistency, readability, or aesthetic appeal across different terminal themes. The modifications are confined to color value adjustments and do not involve structural changes to the configuration file's overall format or layout. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🔇 Additional comments (3)pkg/wconfig/defaultconfig/termthemes.json (3)
The updated colors accurately reflect the official One Dark Pro theme palette, improving visual consistency. Also applies to: 35-36, 38-39
The addition of 47% opacity (77 in hex) to the selection background provides good visibility while supporting terminal transparency.
The change maintains theme consistency while adding appropriate transparency. Please verify the visual appearance of the selection with this color in high-contrast scenarios: ✅ Verification successfulSelection background color change looks good and doesn't conflict with other themes The verification confirms that the warm yellow selection background color 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Check if there are any other files containing color definitions that might conflict
rg -g '!termthemes.json' '#B7950B' .
Length of output: 38 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
fix some more of the selection background colors and fix onedarkpro (was too washed out and not using the correct official colors)