-
Notifications
You must be signed in to change notification settings - Fork 46
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
Question: What's required to update Smile to include latest iOS 11.1 emoji (vampire, mermaid, etc)? #5
Comments
@onmyway133 I went ahead and compiled a list of the new emoji with associated code points and aliases (I did my best to match the conventions already in Smile). https://gist.github.com/kharmabum/0dc6307a0bbccb8a76f93973db00abce |
@kharmabum does my PR #6 fixed the issue? |
@Francescu I would say it's mostly sufficient. The lack of gendered versions of a few of the emoji is regrettable, but you may have had a compelling reason to leave those out. |
I’m not aware of those. Could you list which variations are not working so I can try to fix?
|
@Francescu check out the gist I posted above. |
@kharmabum just tested some of the emojis in the gist (🧙♀️🧙♂️🧛♀️🧛♂️) and they're working fine with Smile. Please not that while some gendered emojis are not present in Emoji.swift they'll work. Feel free to test and do a quick list here of non-working gendered emojis while Smile. Thanks for your help 😊 |
🙏👍
- JC
…________________________________
From: Francescu Santoni <notifications@github.com>
Sent: Monday, February 26, 2018 10:00:12 AM
To: onmyway133/Smile
Cc: Juan-Carlos Foust; Mention
Subject: Re: [onmyway133/Smile] Question: What's required to update Smile to include latest iOS 11.1 emoji (vampire, mermaid, etc)? (#5)
@kharmabum<https://github.com/kharmabum> just tested some of the emojis in the gist (🧙♀️🧙♂️🧛♀️🧛♂️) and they're working fine with Smile. Please not that while some gendered emojis are not present in Emoji.swift<https://github.com/onmyway133/Smile/pull/6/files#diff-2c35896e4293616d55c97d6f56515bbaR1449> they'll work. Feel free to test and do a quick list here of non-working gendered emojis while Smile.
Thanks for your help 😊
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#5 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AAv0xdUVnmpe4qDgD_96AZiOPSTTZ-Mtks5tYvErgaJpZM4QvV-g>.
|
@Francescu @kharmabum Thanks for your contribution ❤️ If you 'd like to be make more changes, I 'd be very happy to add you as contributors to the project |
See: https://blog.emojipedia.org/ios-11-1-emoji-changelog/
The text was updated successfully, but these errors were encountered: