-
Notifications
You must be signed in to change notification settings - Fork 889
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.
The light colors aren't supposed to fit right with the dark main themes and vice versa, so this makes sense. You still need to set these two fields in the new block you added: |
Yeah totally makes sense. I think I'm just tired and not thinking correctly ._. Which icon/text svg should I use though? Since we have multiple options? |
Turn on the |
Head branch was pushed to by a user without write access
This adds the logo icons to the "missing css" which is found when the setting "match top bar with main color" is enabled
Done. |
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.
suggestion (non-blocking): All of the Everforest Light base themes are right at the borderline of what's considered an acceptable color contrast for text on a background per the WCAG AA standard. This functionally makes these themes persona non grata for colorblind users and users with low vision. I recommend that the luminosity values of the light text colors are lowered for these themes, ideally to the WCAG AAA standard of 7:1, to prevent this issue. I marked this as non-blocking because I know some advocates of these themes are against altering them.
@kommunarr i pointed it out before but from the response on it i thought that that was the only issue |
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 |
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.