-
-
Notifications
You must be signed in to change notification settings - Fork 296
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
iOS 9.1 People emoji 🙃 #25
Conversation
🎉 tests pass 🎉 The simple map will be recompiled/updated in the prepublish script, correct? I don't have do do anything to it? |
\o/ !! looks awesome 🎉!! Would love to help. Is there a particular reason you did one commit per emoji? For reference it looks like the official unicode descriptions are here, though it looks like the emojipedia ones are identical. re: the simple map, it does get updated on prepublish, but it's checked into the repo for now (@muan this is up to you obviously) since some folks either find it more accessible or use bower or something like that. It could also just be regenerated and checked in when the lib gets versions. @muan: is there a reason for the Super excited, awesome work 😄 |
@yoshokatana 😍 thank you so much!
I'd go with however it's ordered/categorized in iOS/OSX.
I added that because |
Ahh, makes sense. I think that's fine for now, as long as people know they need to update the keys when they add new emoji. 😅 |
😞 sad that emoji doesn't just order itself correctly in the unicode space! Just to check that I understand correctly, does |
Yup! |
@muan: Are there any other updates that need to happen in this pull request before it's ready to merge? |
Thanks @yoshokatana, I've updated it with the iOS order. Now 6 categories to go!!! |
Added all the
people
emoji, 🕵found here:Part of #24, but there are still a lot to go. Anyone want to help? 🤘
(Note: I wasn't sure how to order them, so I ended up putting them where I thought they'd logically go. Not important for accessing the library, but might help/hinder maintainability of the list)