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

Add Everforest color themes #6468

Open
wants to merge 19 commits into
base: development
Choose a base branch
from

Conversation

DontBlameMe99
Copy link
Contributor

@DontBlameMe99 DontBlameMe99 commented Dec 28, 2024

Add the everforest color themes

Pull Request Type

  • Bugfix
  • Feature Implementation
  • Documentation
  • Other

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:

  • Six base theme variants:
    • Everforest Dark Hard
    • Everforest Dark Medium
    • Everforest Dark Low
    • Everforest Light Hard
    • Everforest Light Medium
    • Everforest Light Low
  • Fourteen main/secondary color themes
    • Everforest Dark Red
    • Everforest Dark Orange
    • Everforest Dark Yellow
    • Everforest Dark Green
    • Everforest Dark Aqua
    • Everforest Dark Blue
    • Everforest Dark Purple
    • Everforest Light Red
    • Everforest Light Orang
    • Everforest Light Yellow
    • Everforest Light Green
    • Everforest Light Aqua
    • Everforest Light Blue
    • Everforest Light Purple

I have also already compressed the SVG files on the website https://svgoptimizer.com

Screenshots

image
image
image
image

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.

DontBlameMe added 3 commits December 28, 2024 22:26
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.
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) December 28, 2024 21:46
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Dec 28, 2024
@efb4f5ff-1298-471a-8973-3d47447115dc

Everforest Dark Low
Everforest Light Low

The official themes use Soft instead of Low in the name. Any reason this is changed to Low?

@DontBlameMe99
Copy link
Contributor Author

Everforest Dark Low
Everforest Light Low

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?
image

@efb4f5ff-1298-471a-8973-3d47447115dc

Everforest Dark Low
Everforest Light Low

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
the pallete got introduced on Dec 7, 2022

Best to stick with the most recent one so no changes needed :)

@efb4f5ff-1298-471a-8973-3d47447115dc

im not seeing Turquoise anywhere in the pallete

@DontBlameMe99
Copy link
Contributor Author

DontBlameMe99 commented Dec 29, 2024

im not seeing Turquoise anywhere in the pallete

I took a look at those colors:
image
and I named them. I just found out (by reading - I don't know why I didnt do this before) that this color (#83C092) is called aqua. I assumed it would be called turquoise.
Should I change the name to aqua?

Found here:
image

@efb4f5ff-1298-471a-8973-3d47447115dc

im not seeing Turquoise anywhere in the pallete

I took a look at those colors: image and I named them. I just found out (by reading - I don't know why I didnt do this before) that this color (#83C092) is called aqua. I assumed it would be called turquoise. Should I change the name to aqua?

Yes please change it to the official used name

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added U: Waiting for Response from Author and removed PR: waiting for review For PRs that are complete, tested, and ready for review labels Dec 29, 2024
auto-merge was automatically disabled December 29, 2024 00:52

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.
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) December 29, 2024 00:52
auto-merge was automatically disabled December 29, 2024 00:55

Head branch was pushed to by a user without write access

This keeps the a-z sorting in the ThemeSettings.vue file
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) December 29, 2024 00:55
Changes the css I have missed from turquoise to aqua
auto-merge was automatically disabled December 29, 2024 00:57

Head branch was pushed to by a user without write access

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) December 29, 2024 00:57
@DontBlameMe99
Copy link
Contributor Author

@efb4f5ff-1298-471a-8973-3d47447115dc I have implemented what you have asked for.

auto-merge was automatically disabled December 29, 2024 01:11

Head branch was pushed to by a user without write access

This also applies the a-z sorting in the colors.js files.
@DontBlameMe99
Copy link
Contributor Author

Need to love that the color scheme itself just doesn't have good contrast :D
Fixed, @kommunarr / @efb4f5ff-1298-471a-8973-3d47447115dc

@kommunarr
Copy link
Collaborator

Thanks @DontBlameMe99, could you post the color contrast links / screenshots so we could both more quickly validate?

kommunarr
kommunarr previously approved these changes Jan 28, 2025
auto-merge was automatically disabled January 28, 2025 16:03

Head branch was pushed to by a user without write access

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) January 28, 2025 16:03
Updates the colors in the colors.js file to match the ones in the colors.css file
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@DontBlameMe99
Copy link
Contributor Author

Light low still has some issues but maybe they are acceptable? CC: @kommunarr

* https://coolors.co/contrast-checker/cf59aa-e5dfc5

* https://coolors.co/contrast-checker/d55d0f-e5dfc5

* https://coolors.co/contrast-checker/25976c-e5dfc5

Oh its needed to check the side nav color as well? I thought only the bg one?

@kommunarr
Copy link
Collaborator

kommunarr commented Jan 28, 2025

Oh its needed to check the side nav color as well?

I don't believe so - although I'm realizing your index.js color choices don't seem to correspond to the right colors, which may be what caused the confusion. They should correspond to the bg-color as we note here:

// Uses the --bg-color for each corresponding theme

auto-merge was automatically disabled January 29, 2025 06:46

Head branch was pushed to by a user without write access

@DontBlameMe99
Copy link
Contributor Author

Oh its needed to check the side nav color as well?

I don't believe so - although I'm realizing your index.js color choices don't seem to correspond to the right colors, which may be what caused the confusion. They should correspond to the bg-color as we note here:

// Uses the --bg-color for each corresponding theme

Thanks for pointing this outt It has been fixed.

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) January 29, 2025 06:46
@efb4f5ff-1298-471a-8973-3d47447115dc

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

auto-merge was automatically disabled January 30, 2025 19:15

Head branch was pushed to by a user without write access

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) January 30, 2025 19:15
Makes the medium dark nav bar color match the other theme variants
@DontBlameMe99
Copy link
Contributor Author

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

Thanks for pointing this out. I somehow messed up the color when creating this variant. It has now been fixed!

@efb4f5ff-1298-471a-8973-3d47447115dc
Copy link
Member

efb4f5ff-1298-471a-8973-3d47447115dc commented Jan 30, 2025

Offtopic Dracula seems to have the same issue? If it even is an issue?

VirtualBoxVM_YRUo62AYjl

Might be an assumption that the side bar is part of Selection?

firefox_VWvXrRqXBT

I think that i will create a PR for this as no other theme does this except for Black but that is just a weird theme overall.

Copy link
Member

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc left a 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>
auto-merge was automatically disabled January 31, 2025 04:27

Head branch was pushed to by a user without write access

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) January 31, 2025 04:28
@DontBlameMe99
Copy link
Contributor Author

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

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request]: Add Everforest themes and variants
4 participants