Skip to content

Admonition block dark mode bug #1519

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

Closed
davydog187 opened this issue Feb 20, 2022 · 5 comments
Closed

Admonition block dark mode bug #1519

davydog187 opened this issue Feb 20, 2022 · 5 comments

Comments

@davydog187
Copy link
Contributor

image

Adding this from my phone but looks like the highlight on text in the blocks needs adjustment

@josevalim
Copy link
Member

@davydog187 it is the new spoiler tag! 😅

@josevalim
Copy link
Member

I have pushed a fix but I have asked the designer to see if a dark mode variation of the blocks make sense. /cc @milmazz

@milmazz
Copy link
Member

milmazz commented Feb 20, 2022

@josevalim Thanks for taking care of this. Once you have the feedback from the designer, please let me know. Then, I can apply the changes to the dark theme.

@sorentwo
Copy link

sorentwo commented Feb 22, 2022

Screen Shot 2022-02-22 at 3 58 37 PM

Here's another example with the admonition in dark mode. Aside from it being really bright, the link (at the end) lacks contrast.

I realize there's a feedback cycle underway with a designer and I wanted to add this for consideration. If you'd prefer a short-term fix for the link contrast, I'm happy to submit one.

@josevalim
Copy link
Member

I fixed links (sorry, I saw your edit just now). I have already asked the designer, I will open up a new issue once I have the new mockups. :)

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

4 participants