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

Adds Quasar #3144

Merged
merged 11 commits into from Jun 9, 2020
Merged

Adds Quasar #3144

merged 11 commits into from Jun 9, 2020

Conversation

Berkmann18
Copy link
Contributor

@Berkmann18 Berkmann18 commented May 31, 2020

Issue: fix #3143

Checklist

  • I updated the JSON data in _data/simple-icons.json
  • I optimized the icon with SVGO or SVGOMG
  • The SVG viewbox is 0 0 24 24

Description

Source URL: https://github.com/quasarframework/quasar-art/blob/cbbbb4b0b7ec7181dfc2d1b29a1ce025e71575bc/src/quasar-logo.svg

@ericcornelissen ericcornelissen added the new icon Issues or pull requests for adding a new icon label Jun 1, 2020
@ericcornelissen
Copy link
Contributor

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.

icon preview

If you have any questions, feel free to ask here 🙂

@Berkmann18
Copy link
Contributor Author

I believe I fixed the issues and set it so it complies with the standards.

@PeterShaggyNoble
Copy link
Member

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.

@Berkmann18
Copy link
Contributor Author

@PeterShaggyNoble TIL it was possible while still having the shapes not touching each other.
I've done that now.

@PeterShaggyNoble
Copy link
Member

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. The markup of the SVG should be just a single line, with no newlines or tabs.
  2. For files sourced from Github, the source URL in the JSON should be the link to the file used - see the "Source Guidelines" section of our contributing guidelines for more information.
  3. From their website, my initial expectation would be that the brand colour should be a grey - they use #263238 in their logos - but I could also see a case being made for the #1976d2 blue used both in their logo and on their website.

@Berkmann18
Copy link
Contributor Author

Berkmann18 commented Jun 2, 2020

1/2. Done.

  1. So what do you recommend? Should I change the hex in the JSON file to #263238 or use one of the blue shades (#42a5f5/#1976d2)?

@PeterShaggyNoble
Copy link
Member

PeterShaggyNoble commented Jun 2, 2020

Thanks for the updates, @Berkmann18.

Your path, however, is coming in at 23.942 wide, rather than 24. And, when comparing it with the first icon in the source file, I'm seeing quite a few differences - which version did you base your SVG off?

You also managed to delete the title tag from the SVG, which is why the build is failing this time 🙊

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.

@Berkmann18
Copy link
Contributor Author

Your path, however, is coming in at 23.942 wide, rather than 24. And, when comparing it with the first icon in the source file, I'm seeing quite a few differences - which version to you base your SVG off?

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).

You also managed to delete the title tag from the SVG, which is why the build is failing this time

It seems that it's due to SVGOMG removing that (not the first time it happens, it seems).

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 [...]

Sure.

@PeterShaggyNoble
Copy link
Member

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?

It seems that it's due to SVGOMG removing that

You can disable that setting in the sidebar; it's the fifth option from the bottom.

@PeterShaggyNoble PeterShaggyNoble added the awaiting reply Issues or pull requests awaiting reply from an individual before it may be addressed label Jun 3, 2020
@Berkmann18
Copy link
Contributor Author

@PeterShaggyNoble The one you see when you use the source URL: quasarframework/quasar-art@9127036.
It's not based off some old version.

You can disable that setting in the sidebar; it's the fifth option from the bottom.

Oh, I see, thank you.

@ericcornelissen
Copy link
Contributor

Just chiming in here with two comments:

  • I would say blue is the brand colour more so than grey, look e.g. at their Medium page. Based on that page I would go with the darker blue.
  • Didn't check if the SVG looks good, but it does need to be centered vertically into the viewbox (but I recommend you don't update that until we checked the SVG @Berkmann18).

@Berkmann18
Copy link
Contributor Author

Berkmann18 commented Jun 4, 2020

I would say blue is the brand colour more so than grey, look e.g. at their Medium page. Based on that page I would go with the darker blue.

According to the colour picker extension I use, it seems to be a different shade (#0273d4) which isn't used in the logo colours or as the solid colour on https://quasar.dev/start/pick-quasar-flavour (where #1976d2 is used as the main blue shade, as shown by the menu bar).

Didn't check if the SVG looks good, but it does need to be centered vertically into the viewbox

Good point, you're right, it's vertically off judging by how it shows up on Inkscape. Quasar

@ericcornelissen
Copy link
Contributor

I would say blue is the brand colour more so than grey, look e.g. at their Medium page. Based on that page I would go with the darker blue.

I would stick with the blue (#1976D2) found in the SVG file you linked (outside the viewbox).

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?

The one you see when you use the source URL

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).

comparison

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.)

@ericcornelissen ericcornelissen removed the awaiting reply Issues or pull requests awaiting reply from an individual before it may be addressed label Jun 4, 2020
@PeterShaggyNoble
Copy link
Member

Agreed @PeterShaggyNoble?

Yup, based on the new evidence presented, I'm happy to go with that choice of colour, too.

@Berkmann18
Copy link
Contributor Author

That is very interesting! For starters, that version (at least the two from that SVG I tried) is actually taller than it is wide

Yeah, I thought I was scaling it wrong but that explains it.

Also, could you also point out which logo in the SVG you linked exactly you used?

I used the top icon (the big one which is the official logo).

@Berkmann18
Copy link
Contributor Author

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).

Wow, I didn't notice it wasn't in a square dimension.

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.)

I used the top one in the source URL (I.e. the big one with the official icon).

@Berkmann18 Berkmann18 closed this Jun 4, 2020
@Berkmann18 Berkmann18 reopened this Jun 4, 2020
@PeterShaggyNoble
Copy link
Member

Thanks for your efforts on this one, @Berkmann18 - everything looks spot on now 👍

@PeterShaggyNoble PeterShaggyNoble merged commit 540635c into simple-icons:develop Jun 9, 2020
ericcornelissen added a commit that referenced this pull request Jun 14, 2020
# New Icons

- Canonical (#3184)
- Der Spiegel (#3168)
- DHL (#3048)
- Land Rover (#3195)
- Openlayers (#3165)
- Prime (#3010)
- Quasar (#3144)

# Updated Icons

- Apache ECharts (#3166)
- Apache Pulsar (#3181)
- Fur Affinity (#2979)
- Jupyter (#3170)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new icon Issues or pull requests for adding a new icon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Quasar
3 participants