1728746 - Incorrect character backspace behavior for compound emoji

Status: ASSIGNED

(Core :: DOM: Editor, defect)

Description

[Qiu Chaofan]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0) Gecko/20100101 Firefox/91.0

Steps to reproduce:

Compound emoji characters, like 🏳️‍🌈 or 👩‍🚀, pressing backspace won't delete the whole character. For 🏳️‍🌈, backspace once produces 🏳️.

Expected results:

These emojis should be seen as single characters.

Comment 1

[Andrei Purice]

Reproduced on the latest version of Nightly 93.0a1 (2021-09-03), beta 92.0 and release 91.0.2.
Setting up flags, severity and a component for this in order to get the dev team involved.
If you feel it's an incorrect one please feel free to change it to a more appropriate one.
Not a recent regression as this can be reproduced to at least Firefox 70.0.
Not reproducible on Chrome.

Comment 2

[Qiu Chaofan]

Attachment: Bug 1728746 - consider grapheme cluster when deleting backwards

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/af6ad4e82c09
https://hg.mozilla.org/integration/autoland/rev/26f74a120814
consider grapheme cluster when deleting backwards r=masayuki

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/c6b00049a937
https://hg.mozilla.org/integration/autoland/rev/c5e57dfd3937
Revert "Bug 1728746 - consider grapheme cluster when deleting backwards r=masayuki" for causing mochitest failures @ test_bug332636.html

Comment 5

[amarc]

Backed out for causing mochitest failures

• Backout link
• Push with failures
• Failure Log

Comment 6

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Failed within these tests:
https://searchfox.org/firefox-main/rev/54da8f6bfead7871ca89f2cb18323af5f00d9620/editor/libeditor/tests/test_bug332636.html#19,24

The Unicode character is: https://www.compart.com/de/unicode/U+10A0F

And now, similar thing is tested within the new test. So, I think we can just remove these 4 lines and optionally adds this character's case into the new test.