<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>13821</bug_id>
          
          <creation_ts>2007-05-22 11:43:20 -0700</creation_ts>
          <short_desc>Paragraph breaks are too tall when font size is tall</short_desc>
          <delta_ts>2010-09-20 00:47:08 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Text</component>
          <version>523.x (Safari 3)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>OS X 10.4</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>http://just1gb.googlepages.com/font_break.html</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Sridhar Gurivireddy">just1gb</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>abarth</cc>
    
    <cc>mitz</cc>
    
    <cc>nickshanks</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>9427</commentid>
    <comment_count>0</comment_count>
    <who name="Sridhar Gurivireddy">just1gb</who>
    <bug_when>2007-05-22 11:43:20 -0700</bug_when>
    <thetext>When bigger font size is specified for text, paragraph breaks in Safari are too tall.

Example:

&lt;p&gt;&lt;font size=&quot;7&quot;&gt;First paragraph.&lt;/p&gt;
&lt;p&gt;Second paragraph.&lt;/p&gt;&lt;/font&gt;

Repro:
######
1) Launch &quot;http://just1gb.googlepages.com/font_break.html&quot; in all browsers to see the height of gaps between the paragraphs.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9396</commentid>
    <comment_count>1</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2007-05-22 14:18:57 -0700</bug_when>
    <thetext>Confirmed.  The &lt;font&gt; misnesting has thrown off margin collapsing here.
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9332</commentid>
    <comment_count>2</comment_count>
    <who name="">mitz</who>
    <bug_when>2007-05-23 01:09:20 -0700</bug_when>
    <thetext>html5lib says that the DOM should be &lt;p&gt;&lt;font/&gt;&lt;p&gt;&lt;p&gt;&lt;font/&gt;&lt;p&gt;. This is basically why WebKit is failing &lt;http://hixie.ch/tests/adhoc/html/parsing/error-handling/002/007.html&gt; and related tests.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>9337</commentid>
    <comment_count>3</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2007-05-23 01:52:22 -0700</bug_when>
    <thetext>Ugh.  That model results in a proliferation of &lt;font&gt; elements that WebKit currently avoids.  If there are 1000 &lt;p&gt; elements, WebKit reopens only once.  That seems much better to me (ignoring issues of compatibility). :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>88390</commentid>
    <comment_count>4</comment_count>
    <who name="Nicholas Shanks">nickshanks</who>
    <bug_when>2008-08-13 05:22:37 -0700</bug_when>
    <thetext>CCing myself, but would be happier if this never got fixed. :-)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>281421</commentid>
    <comment_count>5</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-09-20 00:47:08 -0700</bug_when>
    <thetext>This is fixed by the HTML5 parser.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>