-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Support Dark Mode for Pytest Docs Website #12290
Comments
We use the Pallets Sphinx theme which doesn't seem to offer a dark mode. It looks like even Pallets has plans to switch to Furo which does. Maybe we should reevaluate #6402 (where we ended up upgrading our theme to upstream Pallets in #6453) and instead adopt Furo? I'd be +1 on that. |
👍 from me too, however I probably won't have time to work on this, but would be happy to review the work if someone does tackle this. |
👍 I'm all for dark mode docs everywhere. I'd like to have a go at this, though I suspect a switch to Furo might mean a few design changes to the site. It would be useful to use this issue to discuss the design if it's the right forum to do so. |
Hey all, I took some time to do a quick conversion from Flask to Furo. Only big design change was with the side-nav turning into a note + the main side-nav now being scrollable. |
* furo theme for docs site * removing duplicate tocs from deprecations and reference pages * removing pallets references in code and config * reverting trainings to sidebar * removed sphinx style and unpinned packaging version * updated styles
Co-authored-by: Sam Jirovec <[email protected]>
Thanks a lot @samjirovec, I tried the dark mode and it looks really good! 🚀 Since this feature has already been merged with commit cbf6bd9, I will close this issue. |
What's the problem this feature will solve?
Reading the documentation in the evening can be quite uncomfortable due to the current (very bright) theme of the docs website.
Describe the solution you'd like
Having the option to turn on/off a dark theme for the docs website.
The text was updated successfully, but these errors were encountered: