Skip to content
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

Unreadable components #2

Open
dartt0n opened this issue Dec 31, 2024 · 3 comments
Open

Unreadable components #2

dartt0n opened this issue Dec 31, 2024 · 3 comments
Assignees

Comments

@dartt0n
Copy link

dartt0n commented Dec 31, 2024

First of all, thank you for porting the theme! However, at the moment, the popup window with the warning appears in a way that is difficult to read, as it is white on a bright yellow background
image

The Vscode version of this theme does not contain this issue
image

I have no experience with Zed Themes yet, but maybe this line could be the problem:

"warning.background": null,

I believe it should be xgray1 color (#262626)

@dartt0n dartt0n changed the title Unreadable warning pop-up Unreadable components Dec 31, 2024
@dartt0n
Copy link
Author

dartt0n commented Dec 31, 2024

Going through debug mode for theme, there are some more issues:

Tinted Yellow Button

image

Any Button while Pressed

image

Covered Tinted Icons Button

image

Filled Blue and Yellow Buttons

image

Dragged window

Original:
image

This region should have border or different background color
image

Example from Quill theme
image

Success & Info Text is the same color as background

image

@izuzak
Copy link
Collaborator

izuzak commented Jan 7, 2025

Many thanks for reporting this, @dartt0n! 🙇 Since you already spent some time looking into this and have VSCode set up (for comparisons) -- would you be interested in opening a PR to adjust the colors? Or at least suggesting the right colors for the things that don't look right? 💭 If not, I should be able to find some time this week to take a look.

@dartt0n
Copy link
Author

dartt0n commented Jan 7, 2025

Hi! I have tried to change the theme and prepare the PR, but I didn't have enough time to finish before the national holidays. I'll definitely try again later when I get back to work properly. So far, I have such findings:

Regarding messages, filled & tinted buttons issue:

I have an idea to use the same combination of colors as in Srcery Vim repository for Lightline plugin:
image
Another idea was to highlight using border color as it is in Quill Theme:
image

Colors here would be:

IMG USAGE NR VAR HEX RGB HSL
cyan Accent BG/Border 6 g:srcery_cyan #0AAEB3 10, 174, 179 182, 89%, 37%
yellow Warning BG/Border 3 g:srcery_yellow #FBB829 251, 184, 41 41, 96%, 57%
green Positive BG/Border 2 g:srcery_green #519F50 81, 159, 80 119, 33%, 47%
red Negative BG/Border 1 g:srcery_red #EF2F27 239, 47, 39 2, 86%, 55%
black Dark FG 0 g:srcery_black #1C1B19 28, 27, 25 40, 6%, 10%
brightwhite Light FG 15 g:srcery_bright_white #FCE8C3 252, 232, 195 39, 90%, 88%
xgray2 Light BG 236 g:srcery_xgray2, g:srcery_xgray2_cterm #303030 48, 48, 48 0, 0%, 19%
white Dark BG 7 g:srcery_white #BAA67F 186, 166, 127 40, 30%, 61%

For dragged window two options:
Either make it semitransparent using:

IMG USAGE NR VAR HEXA RGBA HSLA
black black 0 g:srcery_black #1C1B19CC 28, 27, 25, 204 40, 6%, 10%, 80%

Or a little bit brighter using:

IMG USAGE NR VAR HEX RGB HSL
xgray2 xgray2 236 g:srcery_xgray2, g:srcery_xgray2_cterm #303030 48, 48, 48 0, 0%, 19%

Thank you for improving this theme. Perhaps these colors are not final, because it is difficult to imagine them in head without good visualization. Unfortunately, I have not yet had the opportunity to understand the details of how Zed uses theme json files for displaying specific elements, and my trivial attempts to adjust colors have not been successful (seems like fields "warning", "success", "info" etc does not affect reported above problems).

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

No branches or pull requests

2 participants