Landed again the patches for cleaning bogus `<br>`s up when unnecessary
Masayuki Nakano
Bug 1923251 is now fixed again. The patches clean up bogus `<br>` elements when they become unnecessary with inserting something before them. For example, when you create a new paragraph in `<div contenteditable="true">`, the paragraph has a `<br>` like `<div><br></div>`. The `<br>` is required to make the new paragraph height as one-line-height and allow to put caret in it.
However, the first landing caused a regression (Bug 1925635). When you delete per word immediately before a block boundary, new `<br>` should be inserted to make the collapsible white-space which was followed by the deleted word. However, it was not inserted and it looked like the white-space is also removed with the word. The `<br>` was inserted by the post processor of each edit, but touching it may lead regressions and needs to make the various conditions complicated. Therefore, I rewrote each deletion handler inserts `<br>` instead of the post processor.
So, you might see unexpected deletion of trailing white-space after deleting last things at end of paragraph but only in specific conditions. If you meet such regressions, please file a new bug within "DOM: Editor" of "Core" as soon as possible.
I'll start landing some other patches which depend on the landed patches after a couple of days. Then, it becomes harder to back out the patches even for serious cases.
Thanks in advance.
-- Masayuki Nakano <masa...@d-toybox.com> Working on DOM, Events, editor and IME handling for Gecko
Andrew Overholt
--
You received this message because you are subscribed to the Google Groups "dev-pl...@mozilla.org" group.
To unsubscribe from this group and stop receiving emails from it, send an email to dev-platform...@mozilla.org.
To view this discussion visit https://groups.google.com/a/mozilla.org/d/msgid/dev-platform/d36f361b-7988-4d53-b972-645f5248d3b1%40d-toybox.com.