-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Resolved Issue 6316: Use SVG logo and icons instead of png #6491
base: master
Are you sure you want to change the base?
Resolved Issue 6316: Use SVG logo and icons instead of png #6491
Conversation
Thank you! It does look like the logo has got a bit bigger in the process, which is perhaps fine, but intuitively when doing such a change I would try to preserve the appearance as much as possible. By introducing a visual difference, you are making your PR vulnerable to people with opinions about visual design and that's an easy way to get your contribution stuck (since tastes can vary so widely!) Here are some before/after screenshots: BeforeAfterI noticed that the SVG file you introduced has quite some margins at the top and bottom. Is that intentional? Perhaps it would be more convenient to crop those margins. Also I have noticed there are some other uses of the PNG logo that should be converted in the same go: in Also I wonder if we should delete the PNG file in this PR? Or leave it for a while in case some extensions rely on its presence to include it in their own UI? No strong feelings about it. |
Hi @JasonZelmanovich, are you still working on this? |
Yes, I plan to fix those issues by end of the day |
@JasonZelmanovich I have opened a broader issue about removing all uses of raster icons in the tool: #6538.
Let me know if there is anything I can help you with here :) |
@JasonZelmanovich are you still working on this PR or should we close it? |
Fixes #6316