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 Norton icon #7708

Merged
merged 6 commits into from Oct 6, 2023
Merged

Add Norton icon #7708

merged 6 commits into from Oct 6, 2023

Conversation

halacoglu
Copy link
Contributor

norton

Norton is an anti-virus or anti-malware software product, developed and distributed by NortonLifeLock since 1991 as part of its Norton family of computer security products.

Similarweb rank: 2,128

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

  • I retrieved the logo from the official website of Norton and edited it according to Simple Icons requirements.
  • Norton doesn't provide brand guidelines so I used the yellow of the official logo which is #FFE01A.

@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Aug 27, 2022
Copy link
Contributor

@sisp sisp 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 your contribution, @halacoglu! 🙏

I see no notable difference between the source image and your derived icon: 👍

I have one request for change regarding the source URL though.

{
"title": "Norton",
"hex": "FFE01A",
"source": "https://static.nortoncdn.com/static/ngpcdn/static/ngp2022.3.14.8/dist/images/norton-black.svg"
Copy link
Contributor

Choose a reason for hiding this comment

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

From where exactly did you extract this URL? I only see the the logo in the navbar of https://us.norton.com/, and the file URL there is https://us.norton.com/content/dam/norton/logo/logo_norton_d.svg.

According to the source guidelines of the contribution guide:

If the SVG is sourced from:
...

  • Company website: If the SVG is found on the company website (but there is no branding page) the source URL should link to a common page, such as the home page or about page, that includes the source image and not the image file itself.
    ...

For this reason, please update the source URL to the website on which includes the source image instead of using the URL of the image file.

_data/simple-icons.json Outdated Show resolved Hide resolved
icons/norton.svg Show resolved Hide resolved
Co-authored-by: Adam Rusted <hello@adamrusted.me>
@PratyushJayachandran
Copy link
Member

@adamrusted Can you give another look

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.

LGTM, thanks @halacoglu and @PratyushJayachandran!

@adamrusted adamrusted merged commit 9fd3131 into simple-icons:develop Oct 6, 2023
3 checks passed
mondeja added a commit that referenced this pull request Oct 8, 2023
# New Icons

- Arc (#9569)
- Codeium (#9644)
- DJI (#9547)
- NextUI (#9669)
- Norton (#7708)
- OpenText (#7846)
- Studyverse (#9612)
- Turso (#9686)
- Void Linux (#9394)

# Updated Icons

- Hack Club (#9628)
- Hack The Box (#9628)
- HackerOne (#9628)
- HackerRank (#9628)
- Handlebars.js (#9628)
- Handshake (#9628)
- Heroku (#9628)
- Hetzner (#9628)
- Patreon (#9689)
- Yoast (#9398)
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.

None yet

4 participants