1690502 - contentEditable cursor displayed in wrong place when at end of text node, after newline

Status: RESOLVED DUPLICATE of bug 1712834

(Core :: DOM: Editor, defect)

Description

[Marijn Haverbeke]

Setting the selection (with JavaScript) in an element like <pre contenteditable=true>one\n<br></pre> (where the \n is a plain newline) at offset 4 in the one\n text node causes it to be displayed on the line with one. (Setting it at offset 1 in the <pre> node does produce the expected visual cursor position.)

Attached HTML file demonstrates the issue.

Comment 1

[Marijn Haverbeke]

Attachment: test.html

(Somehow, attaching files directly on bug creation either doesn't work, or is confusing me. This should actually add the file.)

Comment 2

[Henri Sivonen (:hsivonen)]

Kagami, can you take a look, please?

Comment 3

[Kagami Rosylight [:saschanaz] (they/them)]

Attachment: Bug 1690502 - Do not ignore the newline followed by <br> in <pre> r=emilio

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/73d1d559436a
Do not ignore the newline followed by <br> in <pre> r=emilio

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/27562 for changes under testing/web-platform/tests

Comment 6

[Kagami Rosylight [:saschanaz] (they/them)]

Oh no, that fix is about bug 1691574. Let's leave this open.

Comment 7

[Phabricator Automation]

Comment on attachment 9202026 [details]
Bug 1690502 - Do not ignore the newline followed by <br> in <pre> r=emilio

Revision D104523 was moved to bug 1691574. Setting attachment 9202026 [details] to obsolete.

Comment 8

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot

Comment 9

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/73d1d559436a

Comment 10

[Kagami Rosylight [:saschanaz] (they/them)]

Fixed in: