hfg-gmuend / openmoji

Open source emojis for designers, developers and everyone else!
http://openmoji.org
Creative Commons Attribution Share Alike 4.0 International
3.84k stars 213 forks source link

Transferring 16 emojis, mainly Spain subdivision flags (April 2024) #495

Closed xavizardKnight closed 2 months ago

xavizardKnight commented 2 months ago

Heyas! This is the PR for transferring the next batch of 16 emojis that I've been talking in some issues lately, including Spain subdivisions and four more brand icons. This time, I've written the correct month on the branch name, that's already an improvement! xD

Included emojis in this PR:

Captura de pantalla a 2024-04-08 18-46-08

Merging this commit will close issue #416; as with this PR, all the subdivision flags of Spain will be done!

Issue #468 could also be closed. As I've debated in said issue that because of attribution requirements, not all the popular programming icons can be added. This issue implements the Dotnet, Kotlin and Golang icons, which are all public domain as far as I know; the Ruby icon is also here for it's dual meaning of programming language and gemstone.

A few notes:

2024-04-09_00-18

Please let me know if something needs to be changed before merging this PR.

Thank you and have a nice day!

b-g commented 2 months ago

@xavizardKnight Many thanks for this! Wow!

Comments:

Hence besides the "GO", the rest seems to be ready to go. Could you tweak the "GO" a bit further and the merge everything on one go ... or happy to tinker with the GO in a sperate PR so that we can merge the rest asap. Up to you.

xavizardKnight commented 2 months ago

the extras-openmoji.csv seem to be fine, at least here in github and on my machine it looks good. Maybe a local git glitch on your end?

Then probably is a glitch on my end. If everything is fine in your end; then it's okay.

Hence besides the "GO", the rest seems to be ready to go. Could you tweak the "GO" a bit further and the merge everything on one go ... or happy to tinker with the GO in a sperate PR so that we can merge the rest asap. Up to you.

The original Golang icon has the Go letters in a big, wide and round font; maybe we could try recreating said letters instead of using the OpenMoji font. Said original logo font would probably not result in the inside of the G being filled black, and would probably fix the weird outline.

The Golang icon has been the hardest of all the emojis in this PR, I had several ideas for how to do it, and none of those ideas (including the design in this PR) fully satisfy me, so I'm open to exploring more ideas or revisit the old ones I had.

As to better test it without delaying the inclusion of the rest of the emojis here, I'll remove the Golang emoji and its associated metadata from this PR and create a new draft PR with only the Golang emoji. This way we will be able to redesign it more calmly.

xavizardKnight commented 2 months ago

Golang has been removed from this PR.

I'll open a new draft PR after you merge this one, as to avoid potential merge conflicts with the metadata.

b-g commented 2 months ago

@xavizardKnight Great!! Merged! ❤️