Skip to content

style: sun/moon toggle for light/dark mode #12610

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
wants to merge 1 commit into from

Conversation

mcanouil
Copy link
Collaborator

@mcanouil mcanouil commented Apr 23, 2025

Switch the basic toggle for light/dark with a sun and moon toggle.

Screen.Recording.2025-04-24.at.00.01.40.mov

@mcanouil mcanouil self-assigned this Apr 23, 2025
@posit-snyk-bot
Copy link
Collaborator

posit-snyk-bot commented Apr 23, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@cscheid
Copy link
Collaborator

cscheid commented Apr 23, 2025

I'm not merging a PR with an astronomically-impossible crescent, sorry :)

@mcanouil
Copy link
Collaborator Author

mcanouil commented Apr 23, 2025

I would be more concerned about the actual accessibility status of the toggle button 😅
I realised when tweaking it that it's actually not at all accessible 😕 (that's for another PR as it can't be solved using CSS)

@mcanouil
Copy link
Collaborator Author

mcanouil commented Apr 28, 2025

FWIW, I've changed the CSS for my Quarto extensions listing (+ light/dark mode): https://m.canouil.dev/quarto-extensions/

@cscheid
Copy link
Collaborator

cscheid commented Apr 29, 2025

I'm going to close this, but FYI later on in 1.8 we're going to switch to the same animation as the Shiny team uses.

@cscheid cscheid closed this Apr 29, 2025
@mcanouil mcanouil deleted the style/sun-moon-toggle branch April 29, 2025 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants