Bug 13821 - Paragraph breaks are too tall when font size is tall
Summary: Paragraph breaks are too tall when font size is tall
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Text (show other bugs)
Version: 523.x (Safari 3)
Hardware: PC OS X 10.4
: P2 Normal
Assignee: Nobody
URL: http://just1gb.googlepages.com/font_b...
Keywords:
Depends on:
Blocks:
 
Reported: 2007-05-22 11:43 PDT by Sridhar Gurivireddy
Modified: 2010-09-20 00:47 PDT (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.