-
Notifications
You must be signed in to change notification settings - Fork 896
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.
Need to love that the color scheme itself just doesn't have good contrast :D |
Thanks @DontBlameMe99, could you post the color contrast links / screenshots so we could both more quickly validate? |
Head branch was pushed to by a user without write access
Updates the colors in the colors.js file to match the ones in the colors.css file
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.
Light low still has some issues but maybe they are acceptable? CC: @kommunarr
Oh its needed to check the side nav color as well? I thought only the bg one? |
I don't believe so - although I'm realizing your Line 632 in b5b60f1
|
Head branch was pushed to by a user without write access
Thanks for pointing this outt It has been fixed. |
Dark Medium has a darker side bar compared to the other components. You can see that Dark Hard en Low have the same color for almost all components. Is there a particular reason for this? VirtualBoxVM_VIBloYfJRx.mp4 |
Head branch was pushed to by a user without write access
Makes the medium dark nav bar color match the other theme variants
Thanks for pointing this out. I somehow messed up the color when creating this variant. It has now been fixed! |
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.
Maybe im looking at this one too long but it seems like the gaps between these components appears smaller when switching to Light Low. I know that isnt happening but the colors make it look like that. Again im looking at this PR for way too long so this could be a nothing burger
VirtualBoxVM_g0GZWfeORp.mp4
@kommunarr is this concerning?
Co-authored-by: efb4f5ff-1298-471a-8973-3d47447115dc <73130443+efb4f5ff-1298-471a-8973-3d47447115dc@users.noreply.github.com>
Head branch was pushed to by a user without write access
I think this only looks like it because the contrast in there is small. I don't think it's possible for the size to change, since I am not touching any CSS which does that. |
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.