Skip to content
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

Man Juggling 🤹‍♂ from categories does not match emoji as man_juggling #13

Open
joeboyscout04 opened this issue Jun 5, 2018 · 3 comments

Comments

@joeboyscout04
Copy link

There is a strange behavior where the man_juggling from the emojiCategories isn't recognized as an emoji using isEmoji and doesn't any values in the emojiList when I do a CTRL-F in Xcode, even though it's very clearly there in the emojiList if I look visually.

I'm not sure exactly what's wrong or how far it extends to other emoji, but it definitely seems like something isn't right.

@joeboyscout04
Copy link
Author

Looks like there are 48 emoji from the categories which don't match emojiList and then aren't recognized as emoji.

let allCategoryEmoji = emojiCategories.flatMap({ $0.value })
let nonEmojiCategoryEmoji = allCategoryEmoji.filter { return !isEmoji(character: $0)}
print(nonEmojiCategoryEmoji)
// result
["🏋️", "🤼‍♀️", "🤼‍♂️", "🤸‍♀️", "🤸‍♂️", "⛹️", "🤾‍♀️", "🤾‍♂️", "🏌️", "🏄‍♀️", "🏊‍♀️", "🤽‍♀️", "🤽‍♂️", "🚣‍♀️", "🚴‍♀️", "🚵‍♀️", "🤹‍♀️", "🤹‍♂️", "👱‍♀️", "👳‍♀️", "👮‍♀️", "👷‍♀️", "💂‍♀️", "🕵️", "👩‍⚕️", "👨‍⚕️", "👩‍✈️", "👨‍✈️", "👩‍⚖️", "👨‍⚖️", "🙇‍♀️", "💁‍♂️", "🙅‍♂️", "🙆‍♂️", "🙋‍♂️", "🤦‍♀️", "🤦‍♂️", "🤷‍♀️", "🤷‍♂️", "🙎‍♂️", "🙍‍♂️", "💇‍♂️", "💆‍♂️", "👯‍♂️", "🚶‍♀️", "🏃‍♀️", "☄️", "⌚️"]

@onmyway133
Copy link
Owner

@joeboyscout04 Hi, that's because those data are from different sources. I will try to find time to fix, but in the meantime PRs are welcome

@joeboyscout04
Copy link
Author

Sure. Which source is the canonical one, and how do you recommend to fix?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants