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

docs: adds a dark theme compatible svgs, fixes #5492 #5501 #5511

Merged

Conversation

bmartinez287
Copy link
Collaborator

The Issue

relates to #5501 #5492
The #gh-dark-mode-only and #gh-light-mode-only tags don't seem to be working so we are switching to the picture tag which is better since it would read correctly outside of github.

How This PR Solves The Issue

It switches from GitHub specific markdown language to the picture tag.

@bmartinez287 bmartinez287 changed the title Adds darkmode for sponsors and ddev logo. docs: adds a dark theme compatible svgs, fixes #5492 #5501 Nov 8, 2023
@bmartinez287
Copy link
Collaborator Author

I used the markdown lint VsCode plugin from David Anson to style the file so hopefully this covers that issue.

@rfay
Copy link
Member

rfay commented Nov 8, 2023

If you're happy with it move it to "ready for review" and say when you think it can be pulled.

I think https://github.com/ddev/ddev/blob/03abebadc08e5f8606c4fb74f9de9348fab269f1/README.md looks ok normal and also dark.

@bmartinez287 bmartinez287 marked this pull request as ready for review November 8, 2023 17:21
@bmartinez287 bmartinez287 requested a review from a team as a code owner November 8, 2023 17:21
@bmartinez287
Copy link
Collaborator Author

It should be good to go now.

@rfay rfay merged commit 743cd77 into ddev:master Nov 8, 2023
8 of 11 checks passed
@bmartinez287
Copy link
Collaborator Author

Seems like it's working well this time around. Yay!

@rfay
Copy link
Member

rfay commented Nov 8, 2023

Thanks for all your work on this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants