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

Add Qt #3162

Merged
merged 2 commits into from Jun 21, 2020
Merged

Add Qt #3162

merged 2 commits into from Jun 21, 2020

Conversation

PeterShaggyNoble
Copy link
Member

Qt

Issue: Closes #2947, picking up from #2955

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

Icon based on this SVG. I had suggested "The Qt Company" as a possible alternative name in the original PR but, from their brand guidelines, it looks like that would require the inclusion of the wordmark.

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

@ericcornelissen ericcornelissen left a comment

Choose a reason for hiding this comment

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

Everything looks good and your statement

I had suggested "The Qt Company" as a possible alternative name in the original PR but, from their brand guidelines, it looks like that would require the inclusion of the wordmark.

Seems to be correct.

So, I'll be merging this in as is. Thanks for the contribution 🎉

@ericcornelissen ericcornelissen merged commit 0e76841 into simple-icons:develop Jun 21, 2020
github-actions bot added a commit that referenced this pull request Jun 28, 2020
# New Icons

- Cypress (#3228)
- NestJS (#3113)
- Qt (#3162)
- Sennheiser (#3045)

# Updated Icons

- BMC Software (#3239)
- Intercom (#3227)
ericcornelissen pushed a commit that referenced this pull request Jun 28, 2020
- Cypress (#3228)
- NestJS (#3113)
- Qt (#3162)
- Sennheiser (#3045)

- BMC Software (#3239)
- Intercom (#3227)
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.

Add Qt Icon
2 participants