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 Google Messages #2367
Add Google Messages #2367
Conversation
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.
Also see my comment in the issue.
On the topic of this PR, why did you choose "Messages by Google" as title? The app itself is called just "Messages", which is perhaps not a suitable name as it is somewhat generic, but why not "Google Messages"?
I couldn't decide between those two, @ericcornelissen ("Messages" was obviously, as you say, too generic) so went with the one they use in the |
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.
For me this is good to go. Due to potential further discussion about the name (and possibly the colour, if someone has objections) I will leave it to another @simple-icons/maintainers to merge this in (or not) 🙂
I think this is good to merge, thanks for your work on this one @PeterShaggyNoble and for review @ericcornelissen 👍 👍 |
# New icons - Elixir (#2242) - Cinema4d (#2376) - SEAT (#2351) - MAAS (#2391) - Fila (#2405) - Ferrari (#2431) - OpenBSD (#2426) - Zigbee (#2382) - Google Translate (#2430) - LG (#2401) - FIFA (#2404) - Fluentd (#2415) - Google Messages (#2367) - JCB (#2389) - Google Cast (#2352) - Vuetify (#2417) - Spinnaker icon (#2443) - Microsoft SQL Server (#2453, 5c9c622) - pr.co (#2456) - Khronos Group (#2448) - OpenGL (#2450) - Vulkan (#2451) - WebGL (#2452) - WebRTC (#2454) - Symphony (#2468) - Veritas (#2326) - Beatport (#2471) - Hugo (#2455) - Font Awesome (#2447) - pre-commit (#2476) - Aircall (#2466) - Apache ECharts (#2473) - WooCommerce (#2461) - MIDI (#2483) - Serverless (#2484) - Woo (#2479) - Katana (#2497) - Houdini (#2498) - Google Cardboard (#2502) - Nuke (#2496) - Concourse (#2427) - NixOS (#2429) - MariaDB (#2506) - MariaDB Foundation (#2402) - Google Classroom (#2482) - Open Containers Initiative (#2490) - Fastly (#2500) - Peugeot (#2515) - WP Rocket (#2432) - Affinity Photo (#2423) - Google Nearby (#2508) - haveibeenpwned (#2517) - Ferrari Corp (#2360) - Affinity Designer (#2422) - LabVIEW (#2435) - Affinity Publisher (#2424) - Affinity (#2425) - NFC (#2501) # Updated icons - Twitch (#2118) - Pandora (#2428) - Babel (#2499)
Issue: Closes #2357
Checklist
_data/simple-icons.json
viewbox
is0 0 24 24
Description
See #2357 for alternative icon & name options