search

iamcal / emoji-data

Easy to parse data and spritesheets for emoji
MIT License
2.55k stars 301 forks source link

`non_qualified` incorrectly `null` for gendered emoji #233

Closed andersk closed 7 months ago

andersk commented 10 months ago

Ten emoji with multiple U+FE0F codepoints are incorrectly listed with "non_qualified": null, and need to be corrected as follows:

name unified non_qualified
🏋️‍♀️ WOMAN LIFTING WEIGHTS 1F3CB-FE0F-200D-2640-FE0F null → 1F3CB-200D-2640
🏋️‍♂️ MAN LIFTING WEIGHTS 1F3CB-FE0F-200D-2642-FE0F null → 1F3CB-200D-2642
🏌️‍♀️ WOMAN GOLFING 1F3CC-FE0F-200D-2640-FE0F null → 1F3CC-200D-2640
🏌️‍♂️ MAN GOLFING 1F3CC-FE0F-200D-2642-FE0F null → 1F3CC-200D-2642
🏳️‍⚧️ TRANSGENDER FLAG 1F3F3-FE0F-200D-26A7-FE0F null → 1F3F3-200D-26A7
👁️‍🗨️ EYE IN SPEECH BUBBLE 1F441-FE0F-200D-1F5E8-FE0F null → 1F441-200D-1F5E8
🕵️‍♀️ WOMAN DETECTIVE 1F575-FE0F-200D-2640-FE0F null → 1F575-200D-2640
🕵️‍♂️ MAN DETECTIVE 1F575-FE0F-200D-2642-FE0F null → 1F575-200D-2642
⛹️‍♀️ WOMAN BOUNCING BALL 26F9-FE0F-200D-2640-FE0F null → 26F9-200D-2640
⛹️‍♂️ MAN BOUNCING BALL 26F9-FE0F-200D-2642-FE0F null → 26F9-200D-2642

This is due to a simple bug in build_map.php that I fixed a year ago:

Please merge this.

andersk commented 7 months ago

This now affects 46 emoji in v15.1.0. I’ve updated #217. Can you please review this very easy fix?