Skip to content

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