Repository navigation
new icon: keycloak (original) - #2699
ChamikaUluwatta wants to merge 2 commits into
Conversation
There was a problem hiding this comment.
Hey @ChamikaUluwatta, thanks for contributing! Really appreciate your work on this. I noticed that the icon still need some optimization before we can merge the PR. Could you please take a look at the icon optimization guidelines and update them accordingly?
Once that's addressed, we can take another look. Thanks!
Snailedlt
left a comment
Snailedlt
left a comment
There was a problem hiding this comment.
In addition to the suggestions you also need to add at least one version that can be used as a font.
So one of these:
- plain
- plain-wordmark
- line
- line-wordmark
Ask an AI agent to make it for you, or follow the guides in our wiki: https://github.com/devicons/devicon/wiki/
| "original" | ||
| ] | ||
| }, | ||
| "color": "#4482C1", |
There was a problem hiding this comment.
Should probably use a color that's either an official color, or one of the main colors in the logo
There was a problem hiding this comment.
I would choose one of these:
- The blue buttons on their website:
#0d6efd - The main blue color on the icon:
#00b8e3 - The blue links on their website:
#428bca
Co-authored-by: Jørgen Kalsnes Hagen <43886029+Snailedlt@users.noreply.github.com>
Double check these details before you open a PR
developbranch.devicon.jsonfile at the correct alphabetic position as seen hereThis PR closes #1644
Link to prove your SVG is correct and up-to-date.
https://github.com/keycloak/keycloak/blob/8cda7d6f26ac94308a5a7554e2c1124b332db6aa/js/apps/admin-ui/public/favicon.svg?plain=1#L4