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

Replace Riot by Element #4121

Merged
merged 4 commits into from
Nov 23, 2020
Merged

Replace Riot by Element #4121

merged 4 commits into from
Nov 23, 2020

Conversation

ericcornelissen
Copy link
Contributor

@ericcornelissen ericcornelissen commented Nov 23, 2020

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

Closes #3606, removes Riot and replaces it by Element in preparation for the upcoming v4 release.

The new SVG is based on the SVG found in the header of the Element homepage. The color is taken from that SVG. Optimized using the SVGO CLI at a precision of 3 4.

@ericcornelissen ericcornelissen added the breaking change Issues or pull requests that include a breaking change and scheduled for the next major release label Nov 23, 2020
@ericcornelissen ericcornelissen added this to the v4.0.0 milestone Nov 23, 2020
@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Nov 23, 2020
@ericcornelissen ericcornelissen added update icon/data Issues or pull requests regarding icons that are outdated, this can be the SVG or color or both and removed new icon Issues or pull requests for adding a new icon labels Nov 23, 2020
@mondeja
Copy link
Member

mondeja commented Nov 23, 2020

I'm seeing some differences between your optimization and the original, also comparing it with my version:

My optimization

M5.593 14.218c-.485 0-.88-.392-.88-.877 0-3.282 2.668-5.943 5.958-5.943.486 0 .88.393.88.877s-.394.877-.88.877c-2.318 0-4.198 1.877-4.198 4.19 0 .486-.393.878-.88.878zm8.615 4.19c0 .485-.394.878-.88.878-3.29 0-5.956-2.662-5.956-5.945 0-.483.394-.876.88-.876.484 0 .878.393.878.877 0 2.316 1.88 4.192 4.2 4.192.484 0 .878.392.878.877zm4.2-8.626c.484 0 .878.392.878.877 0 3.282-2.667 5.943-5.957 5.943-.486 0-.88-.393-.88-.877s.394-.877.88-.877c2.318 0 4.198-1.877 4.198-4.19 0-.486.393-.878.88-.878zm-8.602-4.19c0-.485.394-.878.88-.878 3.29 0 5.956 2.662 5.956 5.945 0 .483-.394.876-.88.876-.485 0-.878-.393-.878-.877 0-2.316-1.88-4.192-4.2-4.192-.484 0-.878-.392-.878-.877zM12 24c6.627 0 12-5.373 12-12S18.627 0 12 0 0 5.373 0 12s5.373 12 12 12z

Differences between your version (red) and original unoptimized (original color)

orig-eric-fix-1
orig-eric-fix-2

Differences between your version (red) and mine optimized (black)

mondeja-eric-fix-1
mondeja-eric-fix-2

@service-paradis

This comment has been minimized.

@ericcornelissen
Copy link
Contributor Author

Thanks for the review guys, guess I was a little too quick with submitting this one. I re-added the icon, now with an optimized precision of 4

Copy link
Member

@mondeja mondeja left a comment

Choose a reason for hiding this comment

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

Looks good now 👍

@ericcornelissen ericcornelissen merged commit ddb82b2 into simple-icons:develop Nov 23, 2020
@ericcornelissen ericcornelissen deleted the v4/riot-to-element branch November 23, 2020 18:17
@service-paradis service-paradis mentioned this pull request Nov 23, 2020
3 tasks
@ericcornelissen ericcornelissen mentioned this pull request Nov 24, 2020
github-actions bot added a commit that referenced this pull request Nov 29, 2020
…ved icon (v4.0.0)

# New Icons

- GitHub Sponsors (#3730)

# Updated Icons

- AppSignal (#3873)
- Atlassian (#3872)
- Atom (#3884)
- Audio-Technica (#3885)
- Bitbucket (#3887)
- Bower (#3888)
- C (#3868)
- C++ (#3869)

# Renamed Icons

- _Adobe Lightroom CC_ to _Adobe Lightroom_ (#4120)
- _Google Hangouts Chat_ to _Google Chat_ (#4141)
- _Google Hangouts Meet_ to _Google Meet_ (#4141)
- _Groovy_ to _Apache Groovy_ (#4120)
- _Postwoman_ to _Hoppscotch_ (#4122)
- _Riot_ to _Element_ (#4121)
- _Tor_ to _Tor Project_ (#3442)

# Removed Icons

- Mixer (#4119)
ericcornelissen added a commit that referenced this pull request Nov 29, 2020
…ved icon (v4.0.0)

- GitHub Sponsors (#3730)

- AppSignal (#3873)
- Atlassian (#3872)
- Atom (#3884)
- Audio-Technica (#3885)
- Bitbucket (#3887)
- Bower (#3888)
- C (#3868)
- C++ (#3869)

- _Adobe Lightroom CC_ to _Adobe Lightroom_ (#4120)
- _Google Hangouts Chat_ to _Google Chat_ (#4141)
- _Google Hangouts Meet_ to _Google Meet_ (#4141)
- _Groovy_ to _Apache Groovy_ (#4120)
- _Postwoman_ to _Hoppscotch_ (#4122)
- _Riot_ to _Element_ (#4121)
- _Tor_ to _Tor Project_ (#3442)

- Mixer (#4119)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change Issues or pull requests that include a breaking change and scheduled for the next major release update icon/data Issues or pull requests regarding icons that are outdated, this can be the SVG or color or both
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update: Riot has become Element
3 participants