<?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>184308</bug_id>
          
          <creation_ts>2018-04-04 10:14:52 -0700</creation_ts>
          <short_desc>REGRESSION: Excessive table cell height when all rows have height set and rowspan &gt; 1</short_desc>
          <delta_ts>2026-02-01 18:34:07 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Tables</component>
          <version>Safari Technology Preview</version>
          <rep_platform>All</rep_platform>
          <op_sys>macOS 10.12</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=155487</see_also>
    
    <see_also>https://bugs.webkit.org/show_bug.cgi?id=21397</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>BrowserCompat, InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Tal Aloni">tal</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ahmad.saleem792</cc>
    
    <cc>ap</cc>
    
    <cc>bfulgham</cc>
    
    <cc>karlcow</cc>
    
    <cc>ntim</cc>
    
    <cc>rniwa</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>zalan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1411767</commentid>
    <comment_count>0</comment_count>
      <attachid>337183</attachid>
    <who name="Tal Aloni">tal</who>
    <bug_when>2018-04-04 10:14:52 -0700</bug_when>
    <thetext>Created attachment 337183
HTML Test Case

I have confirmed the presence of this bug in the latest Safari technology preview (11.1), as well as Safari 9 and 10 (and probably even earlier versions).
Attached a test case, the expected behavior is for the first table cell to have an height of 479px (as can be confirmed using Firefox &amp; Chrome),
In Safari there is currently a very noticeable excessive height to the cell.

This bug may be related to bug 21397</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1412648</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2018-04-07 16:58:47 -0700</bug_when>
    <thetext>&lt;rdar://problem/39262534&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1412724</commentid>
    <comment_count>2</comment_count>
    <who name="Tal Aloni">tal</who>
    <bug_when>2018-04-08 09:23:30 -0700</bug_when>
    <thetext>This bug is not present in Safari 6.0.5 / WebKit 536.30.1,
This bug is present in Safari 6.1.6 / WebKit 537.78.2</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1412725</commentid>
    <comment_count>3</comment_count>
    <who name="Tal Aloni">tal</who>
    <bug_when>2018-04-08 09:41:51 -0700</bug_when>
    <thetext>This problem is not present in Chrome 49.0.2623.112 / WebKit 537.36</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1892074</commentid>
    <comment_count>4</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2022-08-17 15:47:41 -0700</bug_when>
    <thetext>I am able to reproduce this bug in Safari 15.6.1 and Safari Technology Preview 151 where the &quot;No More Bugs!&quot; table is shown in the center of visual viewport with empty void space on top and bottom, in case of other browsers (Chrome Canary 106 and Firefox Nightly 105), there is no void space on top and the &quot;No More Bugs!&quot; is on top rather than center of page. Just wanted to update latest testing results. Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2052942</commentid>
    <comment_count>5</comment_count>
    <who name="Karl Dubost">karlcow</who>
    <bug_when>2024-08-14 22:24:29 -0700</bug_when>
    <thetext>In Firefox, Chrome 

                      HEIGHT
table ............... 479
 tbody .............. 479 
  tr ................  14   (style=&quot;height:14px;&quot;)
   td rowspan=&quot;32&quot; .. 479   (style=&quot;height:479px;&quot;)
  tr ................  15   (style=&quot;height:15px;&quot;)
     (x 31)

31 x 15 = 465 -&gt; + 14 = 479


In Safari

table ............... 944
 tbody .............. 944 
  tr ................ 479   (style=&quot;height:14px;&quot;)
   td rowspan=&quot;32&quot; .. 944   (style=&quot;height:479px;&quot;)
  tr ................  15   (style=&quot;height:15px;&quot;)
     (x 31)

So we can see there is something very wrong about the computation.

31 x 15   = 465 
465 + 479 = 944 

It&apos;s like it is duplicating the size of something already computed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2052943</commentid>
    <comment_count>6</comment_count>
    <who name="Karl Dubost">karlcow</who>
    <bug_when>2024-08-14 22:32:38 -0700</bug_when>
    <thetext>Maybe something around here. 
https://github.com/WebKit/WebKit/blob/e60e4ad864876164321293a426e6cc34531881f9/Source/WebCore/layout/formattingContexts/table/TableLayout.cpp#L431-L443</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2053072</commentid>
    <comment_count>7</comment_count>
    <who name="Tim Nguyen (:ntim)">ntim</who>
    <bug_when>2024-08-15 11:30:24 -0700</bug_when>
    <thetext>(In reply to Karl Dubost from comment #6)
&gt; Maybe something around here. 
&gt; https://github.com/WebKit/WebKit/blob/
&gt; e60e4ad864876164321293a426e6cc34531881f9/Source/WebCore/layout/
&gt; formattingContexts/table/TableLayout.cpp#L431-L443

That code is for the LFC prototype, not used in production.

You&apos;d probably want to look into Source/WebCore/rendering</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2053182</commentid>
    <comment_count>8</comment_count>
    <who name="Karl Dubost">karlcow</who>
    <bug_when>2024-08-15 18:46:08 -0700</bug_when>
    <thetext>ha thanks. 
19 years old comment.  :) Things might have changed a bit. 
https://github.com/WebKit/WebKit/blame/4e87c5073b3ed5648e47a3ffe1f5128a5fdd81f8/Source/WebCore/rendering/RenderTableSection.cpp#L173-L179</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2177057</commentid>
    <comment_count>9</comment_count>
    <who name="Karl Dubost">karlcow</who>
    <bug_when>2026-02-01 18:34:07 -0800</bug_when>
    <thetext>Still happening in STP 235.

More compact test
https://codepen.io/webcompat/pen/EayEqvG

&lt;style&gt;
  table {
    background-color: gold;
  }
  td {width: 20px;}
  table, tr, td {border:1px solid black;}
&lt;/style&gt;
&lt;table&gt;
  &lt;tr style=&quot;background-color: pink;height: 30px&quot;&gt;
      &lt;td rowspan=&quot;3&quot; style=&quot;height:70px&quot;&gt;
          &lt;div style=&quot;background-color:orange;border: 1px solid red;height: 68px;&quot;&gt;&lt;/div&gt;
      &lt;/td&gt;
  &lt;/tr&gt;
  &lt;tr style=&quot;height: 20px; width:20px;background-color: blue&quot;&gt;&lt;/tr&gt;
  &lt;tr style=&quot;height: 20px; width:20px;background-color: green&quot;&gt;&lt;/tr&gt;
&lt;/table</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>337183</attachid>
            <date>2018-04-04 10:14:52 -0700</date>
            <delta_ts>2018-04-04 10:14:52 -0700</delta_ts>
            <desc>HTML Test Case</desc>
            <filename>TestCase.htm</filename>
            <type>text/html</type>
            <size>2071</size>
            <attacher name="Tal Aloni">tal</attacher>
            
              <data encoding="base64">PCFkb2N0eXBlIGh0bWw+DQo8aHRtbD4NCgk8Ym9keT4NCgkJPHN0eWxlIHR5cGU9InRleHQvY3Nz
Ij4NCgkJCXRhYmxlDQoJCQl7DQoJCQkJdmVydGljYWwtYWxpZ246dG9wOw0KCQkJCWVtcHR5LWNl
bGxzOnNob3c7DQoJCQkJdGFibGUtbGF5b3V0OmZpeGVkOw0KCQkJCWJvcmRlci1zcGFjaW5nOjBw
eDsNCgkJCX0NCgkJCQ0KCQkJdGFibGUgdGQNCgkJCXsNCgkJCQlwYWRkaW5nOjBweDsNCgkJCX0N
CgkJPC9zdHlsZT4NCgkJPHRhYmxlIHN0eWxlPSJ3aWR0aDoxMDAlOyI+DQoJCQk8IS0tIHRyIGhl
aWdodCBpcyBzcGVjaWZpZWQgdG8gd29yay1hcm91bmQgYSBmaXJlZm94IGJ1ZyAtLT4NCgkJCTx0
ciBzdHlsZT0naGVpZ2h0OjE0cHg7Jz4NCgkJCQk8dGQgcm93c3Bhbj0nMzInIHN0eWxlPSdoZWln
aHQ6NDc5cHg7Jz4NCgkJCQkJPGRpdiBzdHlsZT0nYmFja2dyb3VuZC1jb2xvcjojZmZmZmZmO2Jv
cmRlcjpzb2xpZCAxcHggYmxhY2s7IGZvbnQtc2l6ZToxMXB4OyBvdmVyZmxvdzpoaWRkZW47aGVp
Z2h0OjQ3N3B4OyBtaW4taGVpZ2h0OjQ3N3B4Oyc+DQoJCQkJCQk8ZGl2IHN0eWxlPSdoZWlnaHQ6
MTAwJTtib3JkZXItbGVmdDpzb2xpZCA1cHggI2ZmZmZmZjsnPg0KCQkJCQkJCTxkaXYgc3R5bGU9
J2hlaWdodDoxMDAlOyBwYWRkaW5nLWxlZnQ6MnB4OyBib3JkZXItbGVmdDpzb2xpZCAxcHggIzAw
MDAwMDsgbGluZS1oZWlnaHQ6MTJweDsnPg0KCQkJCQkJCQlObyBNb3JlIEJ1Z3MhDQoJCQkJCQkJ
PC9kaXY+DQoJCQkJCQk8L2Rpdj4NCgkJCQkJPC9kaXY+DQoJCQkJPC90ZD4NCgkJCTwvdHI+DQoJ
CQk8dHIgc3R5bGU9J2hlaWdodDoxNXB4Oyc+DQoJCQk8L3RyPg0KCQkJPHRyIHN0eWxlPSdoZWln
aHQ6MTVweDsnPg0KCQkJPC90cj4NCgkJCTx0ciBzdHlsZT0naGVpZ2h0OjE1cHg7Jz4NCgkJCTwv
dHI+DQoJCQk8dHIgc3R5bGU9J2hlaWdodDoxNXB4Oyc+DQoJCQk8L3RyPg0KCQkJPHRyIHN0eWxl
PSdoZWlnaHQ6MTVweDsnPg0KCQkJPC90cj4NCgkJCTx0ciBzdHlsZT0naGVpZ2h0OjE1cHg7Jz4N
CgkJCTwvdHI+DQoJCQk8dHIgc3R5bGU9J2hlaWdodDoxNXB4Oyc+DQoJCQk8L3RyPg0KCQkJPHRy
IHN0eWxlPSdoZWlnaHQ6MTVweDsnPg0KCQkJPC90cj4NCgkJCTx0ciBzdHlsZT0naGVpZ2h0OjE1
cHg7Jz4NCgkJCTwvdHI+DQoJCQk8dHIgc3R5bGU9J2hlaWdodDoxNXB4Oyc+DQoJCQk8L3RyPg0K
CQkJPHRyIHN0eWxlPSdoZWlnaHQ6MTVweDsnPg0KCQkJPC90cj4NCgkJCTx0ciBzdHlsZT0naGVp
Z2h0OjE1cHg7Jz4NCgkJCTwvdHI+DQoJCQk8dHIgc3R5bGU9J2hlaWdodDoxNXB4Oyc+DQoJCQk8
L3RyPg0KCQkJPHRyIHN0eWxlPSdoZWlnaHQ6MTVweDsnPg0KCQkJPC90cj4NCgkJCTx0ciBzdHls
ZT0naGVpZ2h0OjE1cHg7Jz4NCgkJCTwvdHI+DQoJCQk8dHIgc3R5bGU9J2hlaWdodDoxNXB4Oyc+
DQoJCQk8L3RyPg0KCQkJPHRyIHN0eWxlPSdoZWlnaHQ6MTVweDsnPg0KCQkJPC90cj4NCgkJCTx0
ciBzdHlsZT0naGVpZ2h0OjE1cHg7Jz4NCgkJCTwvdHI+DQoJCQk8dHIgc3R5bGU9J2hlaWdodDox
NXB4Oyc+DQoJCQk8L3RyPg0KCQkJPHRyIHN0eWxlPSdoZWlnaHQ6MTVweDsnPg0KCQkJPC90cj4N
CgkJCTx0ciBzdHlsZT0naGVpZ2h0OjE1cHg7Jz4NCgkJCTwvdHI+DQoJCQk8dHIgc3R5bGU9J2hl
aWdodDoxNXB4Oyc+DQoJCQk8L3RyPg0KCQkJPHRyIHN0eWxlPSdoZWlnaHQ6MTVweDsnPg0KCQkJ
PC90cj4NCgkJCTx0ciBzdHlsZT0naGVpZ2h0OjE1cHg7Jz4NCgkJCTwvdHI+DQoJCQk8dHIgc3R5
bGU9J2hlaWdodDoxNXB4Oyc+CQ0KCQkJPC90cj4NCgkJCTx0ciBzdHlsZT0naGVpZ2h0OjE1cHg7
Jz4JDQoJCQk8L3RyPg0KCQkJPHRyIHN0eWxlPSdoZWlnaHQ6MTVweDsnPgkNCgkJCTwvdHI+DQoJ
CQk8dHIgc3R5bGU9J2hlaWdodDoxNXB4Oyc+CQ0KCQkJPC90cj4NCgkJCTx0ciBzdHlsZT0naGVp
Z2h0OjE1cHg7Jz4JDQoJCQk8L3RyPg0KCQkJPHRyIHN0eWxlPSdoZWlnaHQ6MTVweDsnPg0KCQkJ
PC90cj4NCgkJCTx0ciBzdHlsZT0naGVpZ2h0OjE1cHg7Jz4NCgkJCTwvdHI+DQoJCTwvdGFibGU+
DQoJPC9ib2R5Pg0KPC9odG1sPg==
</data>

          </attachment>
      

    </bug>

</bugzilla>