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

Text of code block within a info admonition block is barely legible in darkmode #1867

Closed
garazdawi opened this issue Feb 19, 2024 · 5 comments

Comments

@garazdawi
Copy link
Contributor

See erlang/otp#8151 for an example.

Not 100% sure if it is the text or background that should be updated?

@DavidOliver
Copy link
Contributor

As the light theme has dark-on-light code, I would think light-on-dark for the dark theme. @josevalim?

@josevalim
Copy link
Member

@DavidOliver please pull latest and run mix build. I have created an example page with all admonition blocks so we can take a look at it in dev.

The issue above only happened if there was no language selected, otherwise we should light on dark, so it was an easy fix. But your feedback on the example page is very welcome.

@DavidOliver
Copy link
Contributor

Good good - looks fine to me in the context of the current approach.

I would be up for experimenting with dark admonition block background colours for the dark theme, creating the inverse of the light theme, so that there's less bright tone in general, which I think is what dark theme users tend to prefer, if you'd be interested.

@josevalim
Copy link
Member

It is up to you. :)

@DavidOliver
Copy link
Contributor

Okay. :) I'll have a go at that and submit a PR if I think it's worth considering before looking at #1866.

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

No branches or pull requests

3 participants