1726475 - Make `Backspace` key behavior next to `<hr>` element compatible with the other browsers in Nightly and early beta builds

Status: RESOLVED FIXED

(Core :: DOM: Editor, defect, P3)

Description

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

I forgot to do this. We have a hidden pref for this and two WPTs are tested with enabling it.

• https://searchfox.org/mozilla-central/rev/d3683dbb252506400c71256ef3994cdbdfb71ada/testing/web-platform/meta/input-events/input-events-get-target-ranges-backspace.tentative.html.ini#4
• https://searchfox.org/mozilla-central/rev/d3683dbb252506400c71256ef3994cdbdfb71ada/testing/web-platform/meta/input-events/input-events-get-target-ranges-non-collapsed-selection.tentative.html.ini#2

We should make it enabled by default in Nightly and early beta builds first.

Comment 1

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

Attachment: Bug 1726475 - Make `Backspace` around `<hr>` element compatible with the other browsers r=m_kato!

Currently, Gecko does not delete <hr> element by Backspace from the
following paragraph. Instead, put caret to after <hr> element and then,
another Backspace deletes the <hr>. This is clearer for users, but
incompatible with the other browsers.

The pref was introduced by https://phabricator.services.mozilla.com/D88969.
And let's try to enable it by default for testers. I guess that it's rare
to allow web apps to allow <hr> element into contenteditable. So, I'm
not sure whether this change can work as experiment.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c3438ee0d186
Make `Backspace` around `<hr>` element compatible with the other browsers r=m_kato

Comment 3

[Cristina Cozmuta (:CrissCozmuta)]

https://hg.mozilla.org/mozilla-central/rev/c3438ee0d186