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
docs: adds a dark theme compatible svgs, fixes #5492 #5501
docs: adds a dark theme compatible svgs, fixes #5492 #5501
Conversation
Moved this to "draft" until the upstream PR gets merged. |
Modifies README.md to add the missing logos
853f9bf
to
b4a5956
Compare
@rfay since the other one was merged this should be ready. |
Thanks! |
Looks like is adding both images. I will swap it for the picture tag. https://www.stefanjudis.com/notes/how-to-define-dark-light-mode-images-in-github-markdown/ |
Could you please fix https://github.com/ddev/ddev/actions/runs/6801051592/job/18491253628 markdownlint failure while you're working on it? Not sure where that came from, could be new version of markdownlint upstream. You'll need markdownlint-cli on your machine. |
Thank you |
Modifies README.md to add the missing logos
However, it requires this PR to get merged first ddev/ddev.com#134
Otherwise, the sponsors will break on GitHub's dark theme. Due to the asset not being there.
Fixes #5492
Manual Testing Instructions
Check that the README sponsors load well on a light GitHub theme and a dark theme.
Release/Deployment Notes
No