-
Notifications
You must be signed in to change notification settings - Fork 340
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
Comments
@davydog187 it is the new spoiler tag! 😅 |
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 |
@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. |
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. |
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. :) |
Adding this from my phone but looks like the highlight on text in the blocks needs adjustment
The text was updated successfully, but these errors were encountered: