-
-
Notifications
You must be signed in to change notification settings - Fork 35
feat(style): use high-contrast themes for code blocks #538
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
Conversation
|
🌐 Fork Preview for PR #538 https://pr-538.ddev-com-fork-previews.pages.dev This preview updates automatically when you push changes to your fork. |
|
There's no stopping you! |
rfay
left a comment
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.
If you like it I like it!
tyler36
left a comment
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.
Looks fine to me, although I'm a life-long dark themed user.
I did note that Google Chrome's Lighthouse complained a lot about the codeblocks in light-mode.
Background and foreground colors do not have a sufficient contrast ratio.
|
Thanks @tyler36 ! I switched to |
|
Thank you @stasadev . Lighthouse seems happier with |
|
PR closed. The Cloudflare Pages preview is no longer updated. |
The Issue
I don't like that we have the same Nord color scheme for code blocks in both light/dark modes.
Comments in the Nord theme have poor contrast.
How This PR Solves The Issue
Uses
github-light-high-contrastfor light andgithub-dark-high-contrastfor dark, see https://shiki.style/themesManual Testing Instructions
Before:
https://ddev.com/get-started/ (Linux)
After:
https://pr-538.ddev-com-fork-previews.pages.dev/get-started/ (Linux)
Before:
https://ddev.com/get-started/ (Windows)
After:
https://pr-538.ddev-com-fork-previews.pages.dev/get-started/ (Windows)
Before:
https://ddev.com/blog/podman-and-docker-rootless/#installing-podman
After (note that comments have better visibility):
https://pr-538.ddev-com-fork-previews.pages.dev/blog/podman-and-docker-rootless/#installing-podman
Automated Testing Overview
Related Issue Link(s)
Release/Deployment Notes