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 Duplicati icon #11072

Merged
merged 3 commits into from
Jun 11, 2024
Merged

Add Duplicati icon #11072

merged 3 commits into from
Jun 11, 2024

Conversation

anhthang
Copy link
Contributor

@anhthang anhthang commented May 30, 2024

image

Issue: closes #10836

Popularity metric:

10k ⭐ on Github
https://github.com/duplicati/duplicati

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 extracted from the website, hex value #1E3A8A from the main part of the logo

@simple-icons simple-icons bot added the new icon Issues or pull requests for adding a new icon label May 30, 2024
@anhthang anhthang requested a review from adamrusted June 5, 2024 09:27
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.

Icon looks to be a perfect match now - though this page makes me think we can't include it...

Trademarks. Company’s name and all related stylizations, graphics, logos, service marks and trade names used on or in connection with any Services are the trademarks of Company and may not be used without permission in connection with your, or any third-party, products or services.

Certain Restrictions. The rights granted to you in the Agreement are subject to the following restrictions: (b) you shall not frame or utilize framing techniques to enclose any trademark, logo, or other parts of the Services (including images, text, page layout or form);

Thoughts, @uncenter?

@uncenter
Copy link
Member

uncenter commented Jun 7, 2024

I'd have to agree unfortunately. Wording is a bit strange but definitely seems like we can't.

@adamrusted
Copy link
Member

Before we close this out, did you want to reach out to them, @anhthang? See if we can get permission?

@adamrusted adamrusted added the permission required Permission should be sought out from the trademark owner before including this icon in our project. label Jun 7, 2024
@adamrusted adamrusted mentioned this pull request Jun 7, 2024
@anhthang
Copy link
Contributor Author

anhthang commented Jun 8, 2024

I have sent email to them to request for permission

@anhthang
Copy link
Contributor Author

anhthang commented Jun 8, 2024

@adamrusted I think we got green light to go

Screenshot 2024-06-08 at 11 46 59 PM

@anhthang anhthang requested a review from adamrusted June 10, 2024 23:26
@adamrusted adamrusted removed the permission required Permission should be sought out from the trademark owner before including this icon in our project. label Jun 11, 2024
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.

Perfect match, thanks @anhthang!

@adamrusted adamrusted merged commit ad7feb9 into simple-icons:develop Jun 11, 2024
3 checks passed
@anhthang anhthang deleted the duplicati branch June 11, 2024 08:12
simple-icons bot added a commit that referenced this pull request Jun 16, 2024
# New Icons

- Anki (#10708) (@uncenter)
- Astra (#11118) (@CodeShakingSheep)
- Bruno (#11101) (@CodeShakingSheep)
- comma (#11034) (@21st-centuryman)
- DATEV (#11160) (@CodeShakingSheep)
- Doctrine (#11064) (@CodeShakingSheep)
- Duplicati (#11072) (@anhthang)
- Picrew (#11068) (@anhthang)
- pino (#10852) (@10xLaCroixDrinker)
- Rime (#11181) (@anhthang)
- Tower (#11170) (@anhthang)
- TypeORM (#10939) (@davesagraf)
- Wagmi (#11065) (@anhthang)
- yabai (#10997) (@anhthang)
- YunoHost (#10947) (@CodeShakingSheep)

# Updated Icons

- GNOME (#11155) (@LitoMore)
- Render (#11130) (@adamrusted)
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.

Request: Duplicati
3 participants