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

Import guides dark mode highlight.js colors #372

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Earlopain
Copy link
Contributor

Builds on top of #371, I wasn't sure if that would be accepted.

Rails guides now always have codeblocks themed dark.
Might make sense here as well, but this just uses the same colors.
Since rails guides use scss it wasn't just copy-paste but nothing multiline cursor editing can't handle.

Mostly colors are more softer. Comments have better contrast.

Before:
https://deploy-preview-371--sdoc.netlify.app/classes/activesupport/cache
grafik

After:
https://deploy-preview-372--sdoc.netlify.app/classes/activesupport/cache
grafik

* The current background is _very_ dark
* Hovered links appear white
* Better contrast on code blocks
Rails guides always have codeblocks themed dark.
Might make sense here as well, but this just uses the same colors.

Mostly colors are more softer. Comments have better contrast.
Copy link
Member

@zzak zzak left a comment

Choose a reason for hiding this comment

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

I'm hesitant to review any bigger changes, but this such a huge improvement for anyone using dark mode.

@Earlopain
Copy link
Contributor Author

FWIW, I don't currently plan to do anything else. It's just that the colors have been bugging me and the guides already have something that someone else has put thought into.

I guess the foundation will give this a proper facelift to bring it inline with guides sometime in the future, whenever that may be.

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.

2 participants