Does not match all combinations of emojis that support multiple skin tones

[scriby]

For instance, the regex will match 🧑🏼‍🤝‍🧑🏻 ('\u{1F9D1}\u{1F3FC}\u200D\u{1F91D}\u200D\u{1F9D1}\u{1F3FB}'), but it doesn't match 🧑🏻‍🤝‍🧑🏼 ('\u{1F9D1}\u{1F3FB}\u200D\u{1F91D}\u200D\u{1F9D1}\u{1F3FC}'). It seems to only work if the darker skin tone is on the left side. There are some more emojis that support multiple skin tones coming, so it would be great if this could match against all possible combinations.

[scriby]

It looks like this issue is resolved in Unicode 12.1 where both combinations are in the RGI.

[JuanCarlosC]

@scriby I am still experiencing the same issue where it will only work if the darker skin tone is on the left side. Did you find a solution?

I've used the inject-sequences.js script to update the regular expression based on the data from Unicode v12.1, but still seeing the same issue.

[scriby]

Supporting the darker skin tones on the left was included in Unicode 12.1. So if you’re running against v12 they won’t work.

…

On Tue, Feb 25, 2020 at 8:06 PM JuanCarlos Chavez ***@***.***> wrote: @scriby <https://github.com/scriby> I am still experiencing the same issue where it will only work if the darker skin tone is on the left side. Did you find a solution? I've used the inject-sequences.js script to update the regular expression based on the data from Unicode v12, but still seeing the same issue. [image: Screen Shot 2020-02-25 at 8 03 45 PM] <https://user-images.githubusercontent.com/7284135/75310901-00873680-580a-11ea-995a-581ad1024134.png> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#65?email_source=notifications&email_token=AAGHAVBEPSORDLQ3QWSMZBTREXTCXA5CNFSM4KBP3QTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEM6WY4I#issuecomment-591228017>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAGHAVG3T3OXWJGSYJPESU3REXTCXANCNFSM4KBP3QTA> .

[JuanCarlosC]

@scriby updated my comment above. Yes, I ran this script against 12.1. Are you no longer experiencing the issue?

[scriby]

I ended up stopping using this library since it wasn’t clear when it would be updated, so I don’t know if it works properly or not when ran against 12.1. But it seems like it should work... Maybe double-check your node_modules to make sure it’s running against the right version of Unicode?

…

On Tue, Feb 25, 2020 at 8:52 PM JuanCarlos Chavez ***@***.***> wrote: @scriby <https://github.com/scriby> updated my comment above. Yes, I ran this script against 12.1. Are you no longer experiencing the issue? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#65?email_source=notifications&email_token=AAGHAVHJN7KVIICMXQ3ZFLDREXYPXA5CNFSM4KBP3QTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEM6ZEIY#issuecomment-591237667>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAGHAVEIGBZHGIJGP2UEQY3REXYPXANCNFSM4KBP3QTA> .

[mathiasbynens]

If I understand the issue correctly, it looks like this has already been fixed when I updated to the latest Unicode version:

$ node -p "require('emoji-regex')().exec('\u{1F9D1}\u{1F3FC}\u200D\u{1F91D}\u200D\u{1F9D1}\u{1F3FB}')"
[ '🧑🏼‍🤝‍🧑🏻', index: 0, input: '🧑🏼‍🤝‍🧑🏻', groups: undefined ]

$ node -p "require('emoji-regex')().exec('\u{1F9D1}\u{1F3FB}\u200D\u{1F91D}\u200D\u{1F9D1}\u{1F3FC}')"
[ '🧑🏻‍🤝‍🧑🏼', index: 0, input: '🧑🏻‍🤝‍🧑🏼', groups: undefined ]

@scriby Can you confirm this is now behaving as you expected?

[mathiasbynens]

Closing since I believe this has been resolved. Feel free to reopen if I misunderstood.