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

Update Zapier #10265

Merged
merged 4 commits into from
Jan 12, 2024
Merged

Update Zapier #10265

merged 4 commits into from
Jan 12, 2024

Conversation

Mvlprem
Copy link
Member

@Mvlprem Mvlprem commented Jan 11, 2024

zapier

Similarweb rank: 5,032

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

svg taken from brand page, Hex taken from brand guidelines.

ss1
ss2

@github-actions github-actions bot added the update icon/data Issues or pull requests regarding icons that are outdated, this can be the SVG or color or both label Jan 11, 2024
Copy link
Member

@PeterShaggyNoble PeterShaggyNoble left a comment

Choose a reason for hiding this comment

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

Thanks for this one, @Mvlprem 👍🏻 Your SVG lines up perfectly with the chosen source. However, page 28 of the brand guidelines mentions that they have different logos for use at different sizes. If you're using Figma, would you be able to extract the "TINY" logo on that page and use that instead as it would be better suited to our purposes.

For the colour, I think we should go with the "Zapier Orange" #ff4f00. Even though "Earth" is the first one listed in the brand guidelines, they state that:

This palette is centred on our Zapier Orange

And it's also the first colour listed at https://brand.zapier.com.

And, for the guidelines, you can link directly to the Figma file.

@Mvlprem
Copy link
Member Author

Mvlprem commented Jan 11, 2024

Yes orange is their brand color, but when using only single color for logo i think only the black and white are allowed from Logo guidance pic above which is why i picked black.

As for different logo if your are talking about the Tiny one on page 28 I've compared both tiny and small and i don't see any major differences they align almost perfectly.

Let me know what do you think and i'll make necessary changes if required.

Copy link
Member

@PeterShaggyNoble PeterShaggyNoble left a comment

Choose a reason for hiding this comment

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

If your version lines up with the tiny one then we're good to go on that front. I would like to get a second opinion from one of the other @simple-icons/maintainers on the colour before merging, though; we generally use the brand's colour rather than the logo's.

@service-paradis
Copy link
Member

Since the hex is used to represent the brand color, I would expect it to be Zapier Orange (#ff4f00).

The usage for the logomark looks rather strict about using only black and white when using it in monochrome while also specifying to not change colors:
image

The app icon might be what we should use. Unfortunately, it looks like they omit the brand name below 40px which is not really suitable for us.
image

Copy link
Member

@adamrusted adamrusted left a comment

Choose a reason for hiding this comment

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

Good spot @service-paradis - I would agree we should therefore use the app icon with the text inside of it, perhaps the 40px one? @Mvlprem - would you mind taking another look?

@PeterShaggyNoble
Copy link
Member

@Mvlprem, what's the source file for the updated version? The only SVG I can find is in their Google Drive folder, but it has square corners. If you extracted it from the Figma file, could you share the source here, please?

@Mvlprem
Copy link
Member Author

Mvlprem commented Jan 11, 2024

@Mvlprem, what's the source file for the updated version? The only SVG I can find is in their Google Drive folder, but it has square corners. If you extracted it from the Figma file, could you share the source here, please?

Sorry forgot about the source, I took it from the figma file page 32, 40px
logo-symbol
I didn't updated the source to figma link, if necessary pls update it if svg is good, sorry for the extra work.

Copy link
Member

@PeterShaggyNoble PeterShaggyNoble left a comment

Choose a reason for hiding this comment

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

I'm seeing a tiny difference in all 4 corners, @Mvlprem, with your version in red. Most likely it can be solved by upping the precision by 1.

The imperfections in the letters in the source file are annoying me but they're present in all versions of the logo and icon, so there's not much we can do about it.

Copy link
Member

@PeterShaggyNoble PeterShaggyNoble left a comment

Choose a reason for hiding this comment

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

I'm now seeing some tiny differences in the lettering, such as this one in the "r". But, given how insignificant they are and how riddled with imperfections the source is, I think we can let them slide.

@PeterShaggyNoble PeterShaggyNoble merged commit c6fb28b into simple-icons:develop Jan 12, 2024
3 checks passed
@Mvlprem Mvlprem deleted the Update-Zapier branch January 13, 2024 02:05
mondeja added a commit that referenced this pull request Jan 14, 2024
# New Icons

- 7Zip (#7334)
- Asahi Linux (#10245)
- Babelio (#10232)
- Brenntag (#10215)
- Carlsberg Group (#8819)
- CNET (#10228)
- DaVinci Resolve (#9766)
- Edge Impulse (#10201)
- Esoteric Software (#10280)
- Fubo (#10192)
- Hyperskill (#10226)
- Knex.js (#10236)
- Linksys (#10231)
- Nomad (#9527)
- Orange (#10227)
- Paramount+ (#10248)
- Reactive Resume (#10225)
- Solana (#10246)
- Spine (#10239)
- TickTick (#10224)

# Updated Icons

- .ENV (#10255)
- .NET (#10255)
- /e/ (#10269)
- 1001Tracklists (#10260)
- 1Password (#10256)
- 42 (#10278)
- 500px (#10270)
- Aston Martin (#10253)
- Dailymotion (#10216)
- Duolingo (#10111)
- Instagram (#10153)
- LADA (#10155)
- Minecraft (#10242)
- Nintendo (#10263, #10266)
- Revolut (#10238)
- Zapier (#10265)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
update icon/data Issues or pull requests regarding icons that are outdated, this can be the SVG or color or both
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants