Skip to content

Fix logo in README for dark and light modes#969

Merged
Siyet merged 3 commits intojcrist:mainfrom
luwqz1:patch-1
Apr 11, 2026
Merged

Fix logo in README for dark and light modes#969
Siyet merged 3 commits intojcrist:mainfrom
luwqz1:patch-1

Conversation

@luwqz1
Copy link
Copy Markdown
Contributor

@luwqz1 luwqz1 commented Jan 7, 2026

No description provided.

The width attribute on <source> inside <picture> only accepts integer
pixel values per HTML spec, not percentages. The actual sizing comes
from width="35%" on the fallback <img>.
@Siyet
Copy link
Copy Markdown
Collaborator

Siyet commented Apr 11, 2026

Pushed a tiny follow-up commit dropping the width="35%" attribute from the two <source> tags. Per HTML spec, width on <source> inside <picture> only accepts integer pixel values, not percentages, and the actual sizing was already coming from the fallback <img> anyway. Otherwise the change is exactly what we want for dark/light README rendering. Thanks for the fix!

Copy link
Copy Markdown
Collaborator

@Siyet Siyet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix and for your patience with the long review delay! The dark-mode README has bugged me for a while, glad to get this in.

@Siyet Siyet merged commit 42f9184 into jcrist:main Apr 11, 2026
1 check passed
@luwqz1 luwqz1 deleted the patch-1 branch April 12, 2026 10:55
@Siyet Siyet mentioned this pull request Apr 12, 2026
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