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

Add FusionAuth icon #10776

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Add FusionAuth icon #10776

wants to merge 2 commits into from

Conversation

service-paradis
Copy link
Member

image

Issue: closes #6444

Popularity metric: ~114k

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

@simple-icons simple-icons bot added the new icon Issues or pull requests for adding a new icon label Apr 5, 2024
PeterShaggyNoble
PeterShaggyNoble approved these changes Apr 5, 2024
View reviewed changes
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.

Looks perfect to me 👍🏻 However, there are a couple of points that we should be able to remove in the concave curves of the right & bottom lines and in the bottom left of the keyhole. If not then this can be merged straight in.

@service-paradis
Copy link
Member Author

Looks perfect to me 👍🏻 However, there are a couple of points that we should be able to remove in the concave curves of the right & bottom lines and in the bottom left of the keyhole. If not then this can be merged straight in.

Great! Managed to remove one in the keyhole. The others always resulted in a an unexpected shift...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PeterShaggyNoble PeterShaggyNoble PeterShaggyNoble approved these changes

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: FusionAuth
2 participants
@service-paradis @PeterShaggyNoble