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

Tweak the main website color to be less jarring #222

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Calinou
Copy link
Contributor

@Calinou Calinou commented Dec 21, 2019

Preview

Before

Before

After

After

@xuv
Copy link
Member

xuv commented Jun 7, 2020

Hello @Calinou

Thanks for the proposition. I know this has been a controversial color choice since the beginning of this fork. But this blue is a testament to the first blue colors associated with unvisited links in defauld browser. And I wanted to clearly make a difference with the original colors from the project it is being forked from.

Now, I agree. This can come as "aggressive" on certain screen setup. And I agree with you, it needs to be refined. But I don't this it should be done by "watering down" the color. I'm hoping we can keep a strong blue, for its historical reasons. While having a design that is less strong to the eye. Maybe by changing how this upper banner looks like.

What do you think?

@Calinou
Copy link
Contributor Author

Calinou commented Jun 7, 2020

@xuv Thanks for your feedback 🙂

Would a darker shade of blue be accepted? It might look good enough as to not be too aggressive.

@xuv
Copy link
Member

xuv commented Jun 7, 2020

Sure. We could try. Do you want to make a proposition for a darker blue?

@TechnologyClassroom
Copy link

The page looks very pleasant with the dark reader extention.

The white background is changed to:

body {
    color: rgb(174, 170, 163);
    background-color: rgb(10, 12, 12);
}

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

Successfully merging this pull request may close these issues.

3 participants