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 wgpu #10358

Merged
merged 8 commits into from
May 26, 2024
Merged

Add wgpu #10358

merged 8 commits into from
May 26, 2024

Conversation

elliotwutingfeng
Copy link
Contributor

image

Issue: closes #7636

GitHub stars: 10k

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

The homepage's hero background color is the same as the bulma CSS primary color, which is turquoise #40E0D0.

@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Jan 25, 2024
@adamrusted
Copy link
Member

Do we think adding in the gaps between colour changes would help this icon, @simple-icons/icon-review?

@LitoMore
Copy link
Member

LitoMore commented Feb 7, 2024

We can ask the thier official people since they have GitHub.

@adamrusted adamrusted added pending Issues that are pending because of e.g. a scheduled brand update in discussion There is an ongoing discussion that should be finished before we can continue and removed pending Issues that are pending because of e.g. a scheduled brand update labels Feb 22, 2024
@PeterShaggyNoble
Copy link
Member

We can ask the thier official people since they have GitHub.

Pinging @jimblandy & @cwfitzgerald from the @gfx-rs team for their feedback here.

I myself would be in favour of cut-outs.

@PeterShaggyNoble PeterShaggyNoble added awaiting reply Issues or pull requests awaiting reply from an individual before it may be addressed and removed in discussion There is an ongoing discussion that should be finished before we can continue labels Mar 15, 2024
@service-paradis
Copy link
Member

I'm also in favor of cut-outs. Without any answer, I think we can go forward and adjust if we get any feedback in the future. Do you want to give this a try @elliotwutingfeng?

@cwfitzgerald
Copy link

I'm no graphics designer, so do whatever you see fit that preserves the intent of the logo :)

@PratyushJayachandran
Copy link
Member

I'm also in favor of cut-outs. Without any answer, I think we can go forward and adjust if we get any feedback in the future. Do you want to give this a try @elliotwutingfeng?

I vote for going forward without cut-outs for now. Logo is still recognizable in a badge.

@adamrusted
Copy link
Member

@PratyushJayachandran - other than the lack of gaps, are you happy the icon matches? If so, did you want to merge?

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.

Quick change to the URL used, but other than that this is good to go.

_data/simple-icons.json Outdated Show resolved Hide resolved
@adamrusted adamrusted merged commit cc9a891 into simple-icons:develop May 26, 2024
3 checks passed
@simple-icons simple-icons bot removed the awaiting reply Issues or pull requests awaiting reply from an individual before it may be addressed label May 26, 2024
simple-icons bot added a commit that referenced this pull request Jun 2, 2024
# New Icons

- Apache HBase (#11042) (@CodeShakingSheep)
- Apache Superset (#10871) (@10xLaCroixDrinker)
- Babylon.js (#11004) (@jglamp)
- Coder (#11030) (@anhthang)
- CSS Design Awards (#10540) (@stefanobartoletti)
- Dictionary.com (#11043) (@anhthang)
- Dovetail (#11029) (@anhthang)
- Knip (#11006) (@uncenter)
- LazyVim (#11028) (@21st-centuryman)
- Modal (#11050) (@anhthang)
- nvm (#11041) (@CodeShakingSheep)
- Rclone (#11005) (@CodeShakingSheep)
- Taipy (#11049) (@anhthang)
- wgpu (#10358) (@elliotwutingfeng)

# Updated Icons

- Acura (#11012) (@jglamp)
- Redis (#11019) (@anhthang)
- SoundCloud (#11000) (@aheneneu)
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: wgpu
7 participants