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.

Sign up for GitHub

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

Jump to bottom

Openzfs icon #7052

Merged
merged 7 commits into from Mar 5, 2022
Merged

Openzfs icon #7052

merged 7 commits into from Mar 5, 2022

Conversation

deepmalamittal
Copy link
Contributor

openzfs

Issue:#6575
Alexa rank:420953

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

@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Jan 13, 2022
@adamrusted adamrusted linked an issue Jan 19, 2022 that may be closed by this pull request
Request: OpenZFS #6575
Closed
@jorgeamadosoria
Copy link
Contributor

I would change the source to https://openzfs.org/OpenZFS%20Guidelines.pdf, which is the official source

@jorgeamadosoria
Copy link
Contributor

image

There are some very minor differences in the holes. Perhaps optimizing with precision 4 can help here.

Or maybe extracting the icon from the pdf linked above, that could also help solve the issue.

@deepmalamittal
Copy link
Contributor Author

@jorgeamadosoria Changes have been implemented. Please review whenever you get a chance.
Let me know if some issue still prevails.

@jorgeamadosoria
Copy link
Contributor

@deepmalamittal It's good enough now, thanks!
LGTM.

@jorgeamadosoria jorgeamadosoria merged commit 3886e50 into simple-icons:develop Mar 5, 2022
@github-actions github-actions bot mentioned this pull request Mar 6, 2022
Merged
mondeja added a commit that referenced this pull request Mar 6, 2022
@mondeja
Release 6 new icons (v6.13.0)
c60e16b 
# New Icons

- AJV (#7183)
- data.ai (#7181)
- Konami (#7120)
- OpenZFS (#7052)
- Simplenote (#7170)
- YOLO (#7040)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
new icon Issues or pull requests for adding a new icon
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Request: OpenZFS
2 participants
@deepmalamittal @jorgeamadosoria