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

📝 Make darkmode logo work on github #909

Closed
wants to merge 5 commits into from

Conversation

pypae
Copy link

@pypae pypae commented Aug 5, 2024

This PR makes the new dark mode logo introduced in #907 available in the GitHub README by using <picture> conditionally.

Note: This got quite ugly, because mkdocs material allows to override the system setting for darkmode, while on github we can only use the media query. Feel free to ignore and close.

@pypae pypae changed the title Make darkmode logo work on github 📝 Make darkmode logo work on github Aug 5, 2024
Copy link

github-actions bot commented Aug 5, 2024

📝 Docs preview for commit 8ee3e7e at: https://cd0e35e6.typertiangolo.pages.dev

@svlandeg
Copy link
Member

svlandeg commented Aug 6, 2024

Thanks for the PR, @pypae! We'll review this and get back to you 🙏

@svlandeg svlandeg added p3 feature New feature, enhancement or request labels Aug 6, 2024
@github-actions github-actions bot added docs Improvements or additions to documentation internal labels Aug 14, 2024
Copy link

📝 Docs preview for commit 6172210 at: https://1581077f.typertiangolo.pages.dev

Modified Pages

@svlandeg svlandeg self-assigned this Aug 14, 2024
@svlandeg svlandeg removed docs Improvements or additions to documentation feature New feature, enhancement or request labels Aug 14, 2024
@github-actions github-actions bot added the docs Improvements or additions to documentation label Aug 14, 2024
@svlandeg svlandeg removed the internal label Aug 14, 2024
@github-actions github-actions bot removed the internal label Aug 14, 2024
Copy link

📝 Docs preview for commit dda7134 at: https://7791d569.typertiangolo.pages.dev

Modified Pages

@svlandeg
Copy link
Member

@tiangolo : I think it's probably better if you review this one directly 🙏

@svlandeg svlandeg removed their assignment Aug 21, 2024
@tiangolo
Copy link
Member

Thanks @pypae! I tried a looot of things to make it compatible with both MkDocs and GitHub 😅

As I want to be able to keep both files in sync.

For now, I would prefer to leave it as is, but thanks for the effort! ☕

@tiangolo tiangolo closed this Aug 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation p3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants