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
Adds Quasar #3144
Adds Quasar #3144
Conversation
Hi @Berkmann18, thank you for taking the time to contribute. Please have a look at section 4 and 5 of the Contributing Guidelines to find out how to improve the SVG you added to fix the build errors. Also, from the screenshot below you can see that the SVG does not fit into the 24x24 viewbox. If you have any questions, feel free to ask here 🙂 |
I believe I fixed the issues and set it so it complies with the standards. |
Thanks for the update, @Berkmann18. However, per our contributing guidlines, the SVG should be a single line of code and consist of one, single path. |
@PeterShaggyNoble TIL it was possible while still having the shapes not touching each other. |
Thanks, @Berkmann18 Before we can review the icon, though, can you optimise it by running it through SVGO or SVGOMG, please? There are also a couple of other things that will need to be updated before we can merge this in:
|
1/2. Done.
|
Thanks for the updates, @Berkmann18. Your path, however, is coming in at You also managed to delete the For the colour, I'm going to stick with my first instinct and say the grey but let's get some feedback from the other @simple-icons/maintainers before you update it. |
That's most likely because the original I based myself off has more elements and the icon itself is weirdly aligned. I had to hack it a few times to get the icon to be at the right place and size (without the extra fluff).
It seems that it's due to SVGOMG removing that (not the first time it happens, it seems).
Sure. |
Thanks, @Berkmann18. In order to ensure we're comparing like with like, could you let us know which version of the logo in the source file you based yours off?
You can disable that setting in the sidebar; it's the fifth option from the bottom. |
@PeterShaggyNoble The one you see when you use the source URL: quasarframework/quasar-art@9127036.
Oh, I see, thank you. |
Just chiming in here with two comments:
|
According to the colour picker extension I use, it seems to be a different shade (
|
I would stick with the blue ( The Medium link was just to indicate that blue does in fact seem to be the brand color, and the site you linked seems to also support that. Agreed @PeterShaggyNoble?
That is very interesting! For starters, that version (at least the two from that SVG I tried) is actually taller than it is wide (whereas your SVG is wider than it is tall). Therefore it is quite difficult to make a meaningful comparison. Anyway, here is the original overlayed in red on top of yours (done in Illustrator). In this zip file you can find a not-optimized version of the logo with the outer circle removed (in case you had trouble removing it from the logo) which I used for the above comparison. You can use it as a jumping of point for updating the Pull Request if you want 😄 Also, could you also point out which logo in the SVG you linked exactly you used? (as it contains 16 of them, 13 inside the viewbox an 3 outside.) |
Yup, based on the new evidence presented, I'm happy to go with that choice of colour, too. |
Yeah, I thought I was scaling it wrong but that explains it.
I used the top icon (the big one which is the official logo). |
Wow, I didn't notice it wasn't in a square dimension.
I used the top one in the source URL (I.e. the big one with the official icon). |
Thanks for your efforts on this one, @Berkmann18 - everything looks spot on now 👍 |
Issue: fix #3143
Checklist
_data/simple-icons.json
viewbox
is0 0 24 24
Description
Source URL: https://github.com/quasarframework/quasar-art/blob/cbbbb4b0b7ec7181dfc2d1b29a1ce025e71575bc/src/quasar-logo.svg