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

Use dark backgrounds for admonition blocks in dark theme #1869

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

DavidOliver
Copy link
Contributor

Admonition block heading backgrounds are set to the hues of the light theme and set to 80% opacity so as to mix with the background dark "gray" of the dark theme.

Admonition blocks‘ text, headings and links now rely on theme-specific custom properties.

As discussed in #1867

Light theme (visually unchanged; here for reference)

Mostly light tones for backgrounds.

Screenshot 2024-02-21 at 02-17-23 Admonition — ExDoc v0 31 1

Dark theme: before

Mix of light and dark tones for backgrounds. Potentially harsh on the eyes.

Screenshot 2024-02-21 at 02-16-56 Admonition — ExDoc v0 31 1

Dark theme: after

Mostly dark tones for backgrounds. Hopefully easier on the eyes and more consistent with the light theme.

Screenshot 2024-02-21 at 02-13-55 Admonition — ExDoc v0 31 1

Admonition block heading backgrounds are set to the hues of the light
theme and set to 80% opacity so as to mix with the background dark
"gray" of the dark theme.

Admonition blocks‘ text, headings and links now rely on
theme-specific custom properties.
@josevalim josevalim merged commit e2c9e3a into elixir-lang:main Feb 21, 2024
4 checks passed
@josevalim
Copy link
Member

💚 💙 💜 💛 ❤️

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

Successfully merging this pull request may close these issues.

2 participants