-
Notifications
You must be signed in to change notification settings - Fork 891
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
Add Everforest color themes #6468
base: development
Are you sure you want to change the base?
Conversation
Adds the Everforest color scheme
This compresses the svg file sizes by using https://svgoptimizer.com
Fixes the light secondary accent colors. Previously they were only three letters, which led to unmatching, ugly looking colors.
The official themes use Soft instead of Low in the name. Any reason this is changed to Low? |
Apparently they are using "Soft" on the main page. I only checked the color palette page, where it says low. Any suggestion on which I should use? |
I checked the commit history of both the pallete and the readme. the readme introduced soft, medium and hard on Mar 14, 2021 Best to stick with the most recent one so no changes needed :) |
im not seeing Turquoise anywhere in the pallete |
Head branch was pushed to by a user without write access
Renames the turquoise accent color to aqua, matching the official name of the palette.
Head branch was pushed to by a user without write access
This keeps the a-z sorting in the ThemeSettings.vue file
Changes the css I have missed from turquoise to aqua
Head branch was pushed to by a user without write access
@efb4f5ff-1298-471a-8973-3d47447115dc I have implemented what you have asked for. |
Head branch was pushed to by a user without write access
This also applies the a-z sorting in the colors.js files.
I am unsure if I should change this, since (as you said) it would definitely alter the UX of this theme. I actually already modified some a little bit to get that ~4.5 contrast. |
I'll let others chime in on that, as I'm not in the target audience for themes that have a low color contrast by design (it makes me squint, even as a non-colorblind person). One thing I did just notice retesting: suggestion: For at least the Everforest Light Low, the watched & hover text colors are darker than the non-watched ones. This is unintuitive, as the watched videos are meant to be less visually dominant. This appears to be an issue of the |
@kommunarr Should the dark theme also be pimray darkest -> tertiary lightest? |
Check our dark theme; it should be light->dark for dark themes because it's about highest contrast with background -> lowest contrast with background. |
Head branch was pushed to by a user without write access
Corrects the light color ordering from dark -> light (instead of light -> dark)
@kommunarr this has been fixed. |
I'm still not a fan of the color contrast for the new light themes, so I'd prefer if others chimed in as well on that before providing my approval. |
Also not a fan of it as pointed out before. What actions could we take to resolve this? |
The fix is to decrease the luminance of the insufficiently dark primary, secondary, or tertiary text colors enough to reach 7:1 with the |
This PR is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 14 days. |
So this is what I should do? |
Yep, go ahead! |
I just again took a look at it. Basically this would result in all the colors becoming the same? I don't know about you but for me this would make the theme look really flat and boring - maybe I am using the tools you have provided wrong? |
You can try things out and see how it looks - better accessibility is strictly better than worse accessibility |
Head branch was pushed to by a user without write access
Implemented? I am not sure if this is all that's needed, but it looks okay. |
Thanks! Apologies for this last minute one, but this issue was noticed with the Catppuchin Mocha theme in #6597, and I now see it here: issue (blocking): The author comment identifier is not clearly visible. This is because the |
I see what you are talking about, but I am unable to come up with a solution. If i prioritize the contrast between the Yes, I could make this work by changing the text to something white-ish as well - but then I am just creating another dark theme? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for looking into this - given that this appears intractable to solve without some additional modifications (e.g., adding a border around it for specific themes), I think we will need to address that separately in the future. Similarly, the search-bar-color
is not sufficiently contrasting with the card-bg-color
in many themes, including these new ones and many existing ones.
Add the everforest color themes
Pull Request Type
Related issue
closes #4650
Description
This pull request adds the everforest color themes (found at https://github.com/sainnhe/everforest)
This means that this adds:
I have also already compressed the SVG files on the website https://svgoptimizer.com
Screenshots
Testing
I have checked every screen I can find (if there are any things missing etc.) I have also checked every color contrast I could think of.