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

Documentation theme updates #3670

Merged
merged 3 commits into from
Jan 6, 2025
Merged

Conversation

dopplershift
Copy link
Member

Description Of Changes

With the update to 0.16.0 of pydata-sphinx-theme, we're seeing more of the purple color in the docs, specifically mousing over the header links. It turns out this is because "violet" is the default secondary--change that to a teal that better matches "Unidata colors." Also adds a version of the logo on the main page suitable for dark mode, and with these fixes, I think we can make light/dark mode use the "auto" setting.

@dopplershift dopplershift added Area: Docs Affects documentation Type: Enhancement Enhancement to existing functionality labels Oct 28, 2024
@dopplershift dopplershift added this to the 1.7.0 milestone Oct 28, 2024
@dopplershift dopplershift requested a review from a team as a code owner October 28, 2024 23:58
@dopplershift dopplershift requested review from dcamron and removed request for a team October 28, 2024 23:58
Use the appropriate version depending on the current dark mode setting.
Replace default secondary color with something that better matches
Unidata colors. Also remove and clean up some of the rest of the custom
theming, including making dark mode work.
@dcamron dcamron enabled auto-merge January 6, 2025 22:11
@dcamron dcamron merged commit 82e91cb into Unidata:main Jan 6, 2025
35 checks passed
@dopplershift dopplershift deleted the theme-updates branch January 6, 2025 22:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Docs Affects documentation Type: Enhancement Enhancement to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants