1272438 - contenteditable fontSize changes create nested font tags that ruin line spacing

Status: UNCONFIRMED

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

Description

[Matias]

Attachment: fontSize changes leading to bad line spacing, examples include newlines before and after changing fontSize because both are buggy in different ways

User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.102 Safari/537.36

Steps to reproduce:

1. Start with an empty contenteditable
2. Set the font size to something large, document.execCommand('fontSize', false, 7)
3. Type some text and hit enter to insert a newline at the end
4. Set the font size to something small, document.execCommand('fontSize', false, 1)
5. Type some text and hit enter to insert a newline at the end


Actual results:

The spacing between lines after setting the fontSize to 1 should be small but since the font tag with size 7 is an ancestor the line spacing is massive:

<font size="7">Large<br><font size="1">Small<br></font></font>


Expected results:

The line spacing should be what is expected of a small font instead of the large one. The font tags probably should not be nested to avoid the problem of line spacing getting out of control:

<font size="7">Large<br></font><font size="1">Small<br></font>

Comment 1

[Jens Stutte [:jstutte]]

Bulk-downgrade of unassigned, 4 years untouched DOM/Storage bugs' priority.

If you have reason to believe this is wrong (especially for the severity), please write a comment and ni :jstutte.