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

joeboyscout04 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

@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 commented 6 years ago

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

onmyway133 / Smile

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