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

Description Sridhar Gurivireddy 2007-05-22 11:43:20 PDT

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.

Comment 1 Dave Hyatt 2007-05-22 14:18:57 PDT

Confirmed.  The <font> misnesting has thrown off margin collapsing here.

Comment 2 mitz 2007-05-23 01:09:20 PDT

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.

Comment 3 Dave Hyatt 2007-05-23 01:52:22 PDT

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). :)

Comment 4 Nicholas Shanks 2008-08-13 05:22:37 PDT

CCing myself, but would be happier if this never got fixed. :-)

Comment 5 Adam Barth 2010-09-20 00:47:08 PDT

This is fixed by the HTML5 parser.