-
-
Notifications
You must be signed in to change notification settings - Fork 860
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 Protonmail #278
Add Protonmail #278
Conversation
It's mindblowing how fast you are! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's required to have rounded corners to keep the look with the others icons; how about:
<svg
xmlns="http://www.w3.org/2000/svg"
aria-label="Protonmail"
fill="#fff"
viewBox="0 0 512 512"><rect
width="512" height="512"
rx="15%"
fill="#8A90C7"/><path d="M259 77s-108-6-131 114v79s0 12 23 29c28 17 91 68 108 68s74-51 102-69c23-16 23-28 23-28v-79C361 71 259 77 259 77zm68 159H185v-45c11-57 74-57 74-57s57 0 68 57v45z"/><path d="M259 384l-34-12-97-68v120s0 11 17 11h222c11 0 17-11 17-11V304l-97 69-28 11z"/></svg>
images/svg/protonmail.svg
Outdated
<svg | ||
xmlns="http://www.w3.org/2000/svg" | ||
aria-label="Protonmail" | ||
viewBox="0 0 512 512"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You could save over 20 bytes by removing the spaces, the group and putting the fill=...
into the <svg
tag.
Also, I think it would require the white background rectangle.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it a white background rectangle with a color circle (rounded rect) on top? In that case, I can knock the paths out of the circle if that ends up being smaller.
Yup - if you can make those small changes, we can merge it. Thanks! |
ref #265