Closed Bug 1542098 Opened 6 years ago Closed 23 days ago

A definite table-cell 'height' is used as the percentage basis for its children

Categories

(Core :: Layout: Tables, defect, P5)

defect

Tracking

()

RESOLVED DUPLICATE of bug 1461852
Tracking Status
firefox-esr60 --- wontfix
firefox66 --- wontfix
firefox67 --- fix-optional
firefox68 --- affected

People

(Reporter: alice0775, Unassigned)

References

()

Details

(4 keywords)

Attachments

(3 files)

Steps To Reproduce:

  1. Open https://tenki.jp/sakura/3/

Actual Results:
Image and text overlap

Expected Results:
Text should be placed below image as Chrome and Edge

Attached file Reduced html

Workaround:
remove 'height: 1em;' from css selector '.popular-spots-box li'

Attached file Testcase #2
Attachment #9056089 - Attachment mime type: application/octet-stream → text/html

So, I guess since the height property is just a "minimum height hint"
or something for table cells we shouldn't use it as a percentage basis?

Severity: normal → minor
Component: Layout → Layout: Tables
Keywords: testcase
OS: Unspecified → All
Priority: -- → P4
Hardware: Unspecified → All
Summary: Layout is broken only Firefox. → A definite table-cell 'height' is used as the percentage basis for its children

My memory is that it's more complicated than that -- although I think it's the height of the row that's used as a percentage basis, in rather complicated conditions that were a function of WebKit reverse engineering Gecko which badly reverse-engineered IE. (I'd hoped to redo the table height code in ~2007 to be more faithful to what IE6 did, which made more sense, but it's too late for that now...)

I think the when does/doesn't a table row serve as a basis for percentage heights rules are pretty complicated -- but I'd at least hope it's always the row height that matters and not particular specified table cell heights.

(There's also the question of which specified heights matter as a percentage basis in the first-pass layout that's used to determine the height of the row, which might be different!)

Bulk moving bugs that have been triaged P4 to P5 (P4 is reserved for WPT bugs).

Priority: P4 → P5
Severity: minor → S4

Based on discussion above, I'm pretty sure this is the same as bug 1461852.

Status: NEW → RESOLVED
Closed: 23 days ago
Duplicate of bug: 1461852
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: