1940377 - Make `WhiteSpaceVisiblityKeeper` use an NBSP instead of collapsible ASCII white-space when it's the last character before a block boundary

Status: ASSIGNED

(Core :: DOM: Editor, defect)

Description

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

When you type a white-space at end of a paragraph or the editing host in contenteditable=true whose white-space is normal, nowrap or pre-line, Gecko puts an ASCII white-space (U+0020) and a padding <br>. On the other hand, the other browsers puts an NBSP (U+00A0) without a padding <br>. When it becomes not the last character, Gecko just removes the padding <br>, but the other browsers replaces the NBSP with an ASCII white-space.

The root cause of bug 1925038, bug 1940278 and bug 1938110 is this bug.

Bug 1940278 and bug 1938110 avoids the regressions with hacky patch, but they may break some other web apps in some edge cases. Therefore, I think that we should give this really high priority.

htsai: I think that I should work on this in Q1 with the highest priority. WDYT?

Comment 1

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

Ideally, we should fix bug 1658699 first, but it may require to work on a lot of edge cases. So, I think that we should fix this without the complete fix of bug 1658699. Only the requirement is, working the white-spaces without major problem.

Comment 2

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

(So, the ideal goal here is, we fix this bug in 136.)

Comment 3

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

Okay, I'll enable the new normalizer with adding the code for inserting content.

Comment 4

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

Okay, I finished writing a patch for inserting text. It seems that I'll finish this faster than I've expected.