Bug 13821
| Summary: | Paragraph breaks are too tall when font size is tall | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Sridhar Gurivireddy <just1gb> |
| Component: | Text | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | abarth, mitz, nickshanks |
| Priority: | P2 | ||
| Version: | 523.x (Safari 3) | ||
| Hardware: | PC | ||
| OS: | OS X 10.4 | ||
| URL: | http://just1gb.googlepages.com/font_break.html | ||
Sridhar Gurivireddy
When bigger font size is specified for text, paragraph breaks in Safari are too tall.
Example:
<p><font size="7">First paragraph.</p>
<p>Second paragraph.</p></font>
Repro:
######
1) Launch "http://just1gb.googlepages.com/font_break.html" in all browsers to see the height of gaps between the paragraphs.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Dave Hyatt
Confirmed. The <font> misnesting has thrown off margin collapsing here.
mitz
html5lib says that the DOM should be <p><font/><p><p><font/><p>. This is basically why WebKit is failing <http://hixie.ch/tests/adhoc/html/parsing/error-handling/002/007.html> and related tests.
Dave Hyatt
Ugh. That model results in a proliferation of <font> elements that WebKit currently avoids. If there are 1000 <p> elements, WebKit reopens only once. That seems much better to me (ignoring issues of compatibility). :)
Nicholas Shanks
CCing myself, but would be happier if this never got fixed. :-)
Adam Barth
This is fixed by the HTML5 parser.