Open Bug 1017534 Opened 12 years ago Updated 3 years ago

Empty row spanned by a rowspanning cell gets a nonzero height

Categories

(Core :: Layout: Tables, defect)

29 Branch
x86_64
Windows 7
defect

Tracking

()

People

(Reporter: irena.kull, Unassigned, NeedInfo)

Details

(Keywords: testcase)

Attachments

(2 files)

Attached file table_background.html
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:29.0) Gecko/20100101 Firefox/29.0 (Beta/Release) Build ID: 20140506152807 Steps to reproduce: Open the attachment with Chrome and Firefox and compare the pictures Actual results: The layouts look very different: 1. the background color is supposed to change gradually from green to blue; but in FF the background color is split into lines, though in Chrome it looks properly and homogeneous; 2. in FF the color in the upper part of the screen is not the same for all the cells - in Chrome it is the same (red). Expected results: It should look the same
Component: Untriaged → Layout
Keywords: testcase
Product: Firefox → Core
Summary: wrong background color → wrong background using linear-gradient
> but in FF the background color is split into lines This is the correct rendering given the markup on the page. The rendering in Chrome is wrong. And if you put the page in standards mode, Chrome renders the same way we do, so what you're seeing is a deliberate spec violation in Chrome in quirks mode. In particular, they deliberately incorrectly size the box of the <html> element in quirks mode. You can see this by giving <html> a border in your stylesheet. > in FF the color in the upper part of the screen is not the same for all the cells This is the interesting part of the testcase. Chrome seems to be making the second row zero-height and making the test1 cell the same height as the test2 cell. You can see this by the alignment of the "test1" text. We seem to be giving the extra height of the "test2" cell to the second. Since the second row has no cell in the first column and row backgrounds only paint where there are cells from the row, you get an area where the row background is not visible. There is no spec for which rendering here is correct, I believe, but I agree that there's a discontinuity here in our rendering: if I put a cell into the second row we assign the row heights proportionately to the cell heights of the cells in the first column. We should probably be treating "no cells" as "0 height"... except that there is a lot of content out there that depends on some bizarre aspects of rowspan sizing behavior, so doing that naively may not be web-compatible. David, do you recall why we have the behavior we have for rowspan here?
Status: UNCONFIRMED → NEW
Component: Layout → Layout: Tables
Ever confirmed: true
Flags: needinfo?(dbaron)
Summary: wrong background using linear-gradient → Empty row spanned by a rowspanning cell gets a nonzero height
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: