-
-
Notifications
You must be signed in to change notification settings - Fork 915
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
New google icons #469
New google icons #469
Conversation
images/svg/gmail.svg
Outdated
viewBox="0 0 512 512"><rect | ||
width="512" height="512" | ||
rx="15%" | ||
fill="#fff"/><path fill="#f2f2f2" d="M120 392V151.075h272V392"/><path fill-opacity=".05" d="M256 285L120 392l-4-212"/><path fill="#d54c3f" d="M120 392H97c-12 0-22-10-22-23V143h45z"/><path fill-opacity=".08" d="M317 392h77V159H82"/><path fill="#f2f2f2" d="M97 121h318L256 234"/><path fill="#b63524" d="M392 392h23c12 0 22-10 22-23V143h-45z"/><path fill="none" stroke="#de5145" stroke-linecap="round" stroke-width="44" d="M97 143l159 115 159-115"/></svg> | ||
fill="white"/><path d="M158 391v-140l-82-63V361q0 30 30 30" fill="#4285f4"/><path d="M76 188l82 63v-98l-30-23c-27-21-52 0-52 26" fill="#c5221f"/><path d="M158 251l98 74l98-74v-98l-98 74l-98-74" fill="#ea4335"/><path d="M436 188l-82 63v-98l30-23c27-21 52 0 52 26" fill="#fbbc04"/><path d="M354 391v-140l82-63V361q0 30-30 30" fill="#34a853"/></svg> |
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.
#fff
is shorter than white
images/svg/gmail.svg
Outdated
@@ -1,6 +1,6 @@ | |||
<svg xmlns="http://www.w3.org/2000/svg" | |||
aria-label="Gmail" role="img" | |||
aria-label="google gmail" role="img" |
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 just Gmail
is fine.
images/svg/google_drive.svg
Outdated
@@ -1,8 +1,6 @@ | |||
<svg xmlns="http://www.w3.org/2000/svg" | |||
aria-label="Google Drive" role="img" | |||
aria-label="google drive" role="img" |
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.
Keep the capitalisation, please.
images/svg/google_meet.svg
Outdated
width="512" height="512" | ||
rx="15%" | ||
fill="white"/><path d="M76 106h360v300h-360z" fill="transparent" stroke="white"/><path d="M166 106v90h-90" fill="#ea4335"/><path d="M166 106v90h120v60l90-73v-47q0-30-30-30" fill="#ffba00"/><path d="M286 256l90-73v146" fill="#00832d"/> | ||
<path d="M376 183l42-34c9-7 18-7 18 7v200c0 14-9 14-18 7l-42-34" fill="#00ac47"/> |
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.
These newlines can be removed.
I did the requested change, it should be good now. |
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.
"Google collaborative content tools" should be "Google Docs Editors"
@Forrest-J if you look on https://www.blog.google/press/ you'll see that Google Docs still has the blue icon : |
In some places & historically Google also calls the suite itself "Google Docs Editors" as opposed to "Google Docs", the word processor. See also Wikipedia. |
I didn't know that, I wonder what would be the best way of solving this. I can imagine 4 possible solutions :
I think 3) has the lesser drawback since google shouldn't change the icon again for a while, nor change it multiple times in a short period of time |
I say option 3. I'm also thinking about keeping the older versions as an option. Just in case Google changes its mind. |
We'll keep the older version anyways since it's not |
Also from how I understand it "Collaborative content tools" isn't the proper name but just a descriptor given that it is set in sentence case, whereas "Google Docs Editors" is the proper name. |
images/svg/gmail.svg
Outdated
@@ -3,4 +3,4 @@ aria-label="Gmail" role="img" | |||
viewBox="0 0 512 512"><rect | |||
width="512" height="512" | |||
rx="15%" | |||
fill="#fff"/><path fill="#f2f2f2" d="M120 392V151.075h272V392"/><path fill-opacity=".05" d="M256 285L120 392l-4-212"/><path fill="#d54c3f" d="M120 392H97c-12 0-22-10-22-23V143h45z"/><path fill-opacity=".08" d="M317 392h77V159H82"/><path fill="#f2f2f2" d="M97 121h318L256 234"/><path fill="#b63524" d="M392 392h23c12 0 22-10 22-23V143h-45z"/><path fill="none" stroke="#de5145" stroke-linecap="round" stroke-width="44" d="M97 143l159 115 159-115"/></svg> | |||
fill="#fff"/><path d="M158 391v-140l-82-63V361q0 30 30 30" fill="#4285f4"/><path d="M76 188l82 63v-98l-30-23c-27-21-52 0-52 26" fill="#c5221f"/><path d="M158 251l98 74l98-74v-98l-98 74l-98-74" fill="#ea4335"/><path d="M436 188l-82 63v-98l30-23c27-21 52 0 52 26" fill="#fbbc04"/><path d="M354 391v-140l82-63V361q0 30-30 30" fill="#34a853"/></svg> |
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.
There seems to be an imperfection between the dark and the light red, a subpixel band of white peeking through.
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.
images/svg/google_meet.svg
Outdated
viewBox="0 0 512 512"><rect | ||
width="512" height="512" | ||
rx="15%" | ||
fill="#fff"/><path d="M76 106h360v300h-360z" fill="transparent" stroke="white"/><path d="M166 106v90h-90" fill="#ea4335"/><path d="M166 106v90h120v60l90-73v-47q0-30-30-30" fill="#ffba00"/><path d="M286 256l90-73v146" fill="#00832d"/><path d="M376 183l42-34c9-7 18-7 18 7v200c0 14-9 14-18 7l-42-34" fill="#00ac47"/><path d="M166 406v-90h120v-60l90 73v47q0 30-30 30" fill="#00ac47"/><path d="M76 316v60q0 30 30 30h60v-90" fill="#0066da"/><path d="M76 196h90v120h-90" fill="#2684fc"/></svg> |
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.
It shouldn't happens, definitely a rendering quirk, I'll try to see if there is a way to avoid this when I have a little time. Probably not this weekend, I got a rush for a school project. |
I added the 5 new google logos
closes #458