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

Logo in README Not Best For GitHub Dark Theme #36

Closed
zicklag opened this issue Jan 11, 2022 · 10 comments
Closed

Logo in README Not Best For GitHub Dark Theme #36

zicklag opened this issue Jan 11, 2022 · 10 comments
Assignees

Comments

@zicklag
Copy link

zicklag commented Jan 11, 2022

Hey there, I just noticed that the logo doesn't look quite right on the GitHub dark theme.

Not sure what the best path forward is, but jus thought I'd let you know in case you don't notice it because you're on the light theme.

image

Neat looking project BTW! 👍

@haixuanTao
Copy link
Collaborator

Thanks! Yeah, ahah, I did it really quickly, I should probably do another header.

Thanks for the issue :)

@pixelspark
Copy link
Collaborator

pixelspark commented Feb 9, 2022

Hi,

I tweaked the logo a bit (a square 'avatar' for the org as well as a logo with slogan that works a bit better on black backgrounds). Inkscape SVG attached. If you like it I will update the logos. Should use SVG's anyway of course.
wonnx_slogan_opt
wonnx_avatar
wonnx_slogan
wonnx_avatar_opt

@haixuanTao
Copy link
Collaborator

haixuanTao commented Feb 9, 2022

Screenshot from 2022-02-09 14-30-41

I'm all in to have svgs but the formating of the SVG you put looks weird on my laptop. I'm, on Firefox.

@pixelspark
Copy link
Collaborator

I'm all in to have svgs but the formating of the SVG you put looks weird on my laptop. I'm, on Firefox.

Looks like the SVG references the font I used. I assumed Inkscape would convert that to paths. Attached are new versions, these don't require the font (the characters are now paths in the SVG). Hopefully these render correctly.

wonnx_avatar_opt
wonnx_slogan_opt

@haixuanTao
Copy link
Collaborator

This looks much better.

I think it would be perfect if it could render well in dark mode as well.
Screenshot from 2022-02-09 14-50-00

But if you don't want to bother feel free to update the settings and commit the new logo.

@pixelspark
Copy link
Collaborator

This looks much better.

I think it would be perfect if it could render well in dark mode as well. Screenshot from 2022-02-09 14-50-00

But if you don't want to bother feel free to update the settings and commit the new logo.

I specifically added the white outline so it looks a little better than the old logo,but I agree it is not yet optimal.

With some SVG tricks you could probably make it dynamic (if the browser supports the CSS 'prefers-color-scheme' mechanism? Not sure how wide-spread/compatible that is).

@pixelspark
Copy link
Collaborator

Here's one that should automatically switch to white text when dark mode is enabled!
wonnx_slogan_opt

@haixuanTao
Copy link
Collaborator

haixuanTao commented Feb 9, 2022

The dark version is perfect and the light version seems great when I click on the SVG. But when it is within a comment the SVG looks weird, not sure why.

Screenshot from 2022-02-09 15-11-39

I think that you can try to put it in the README and preview how it render.

@pixelspark pixelspark self-assigned this Feb 9, 2022
@pixelspark
Copy link
Collaborator

Yes, there's something funny going on with caching I suspect. Will commit this image to the repo later today (I already updated the org's logo and social image).

@zicklag
Copy link
Author

zicklag commented Feb 9, 2022

I'm not sure if GitHub might like sanitize the SVG to prevent it from utilizing the browser CSS rules?

Anyway, even with the white outline it's far better than it was before. 👍

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

No branches or pull requests

3 participants