Skip to content

feat: add manifest logo with dark mode display condition#582

Merged
eemeli merged 4 commits intoeemeli:mainfrom
SebConejo:main
Oct 9, 2024
Merged

feat: add manifest logo with dark mode display condition#582
eemeli merged 4 commits intoeemeli:mainfrom
SebConejo:main

Conversation

@SebConejo
Copy link
Copy Markdown
Contributor

Here are the logos :
We need theme to be added to https://eemeli.org/yaml/images/ repo.

manifest-light
manifest-transparent

@eemeli
Copy link
Copy Markdown
Owner

eemeli commented Oct 9, 2024

Hi! Could you file this PR in eemeli/yaml-docs instead? That's where the image needs to go. Note also that GitHub's readme rendering is really limited, and e.g. selecting an image based on the light/dark mode is not possible. This means that you'll need to pick one image to use.

@SebConejo
Copy link
Copy Markdown
Contributor Author

@eemeli I have added the logo in svg format to the yaml-docs repo. So I updated the file here by replacing the png logo by a svg one, with a white background. So that we call only one logo for manifest.

@eemeli eemeli merged commit f2fa108 into eemeli:main Oct 9, 2024
@eemeli
Copy link
Copy Markdown
Owner

eemeli commented Oct 9, 2024

Thank you! This one's much better in dark mode.

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.

2 participants