Closed Bug 1470436 Opened 7 years ago Closed 6 years ago

thead overlaps tbody when using page-break-(after/before) property

Categories

(Core :: Layout, defect)

Product:
Core
Component:
Layout
Version:
60 Branch
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla73
Tracking Flags:
Tracking Status
firefox-esr68 --- wontfix
firefox71 --- wontfix
firefox72 --- wontfix
firefox73 --- fixed

People

(Reporter: kyle, Assigned: MatsPalmgren_bugz)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

index.html
34.32 KB, text/html
Details
Attached file index.html —
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/67.0.3396.87 Safari/537.36 Steps to reproduce: 1. Open attached HTML 2. Print Preview Actual results: I have page-break-after: always on top level div's so I can create multiple pages. Pages that 'overflow' onto multiple pages work correctly, the thead is duplicated on the next page. Pages that do not overflow onto multiple pages have a problem where the thead overlaps the tbody. It seems to be related to the page-break-after property. Worked in FF 52 ESR Does not work in FF60 and 62 The regression seems to appear in FF56, as it renders properly in FF55. Expected results: The thead does not overlap the tbody on any page. Chrome 67 and Edge on Windows 1803 renders properly.
This might be a duplicate of 1404868
It looks like adding some margin or padding to the element that uses page-break-after will fix the regression. I would still like this regression fixed without the use of this workaround.
Could you try to use mozregression to find a more precise regression range?
Status: UNCONFIRMED → NEW
Component: Untriaged → General
Ever confirmed: true
Keywords: regression, regressionwindow-wanted
Product: Firefox → Core
Component: General → Untriaged
Flags: needinfo?(kyle)
Product: Core → Firefox
Last good revision: e6e712904806da25a9c8f48ea4533abe7c6ea8f4 First bad revision: d6bf703c5deaf1e328babd03d5e68ff2a4ffe10e Pushlog: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=e6e712904806da25a9c8f48ea4533abe7c6ea8f4&tochange=d6bf703c5deaf1e328babd03d5e68ff2a4ffe10e
Keywords: regressionwindow-wanted
Blocks: 1308876
Component: Untriaged → Print Preview
Product: Firefox → Core
That revision window seems to the same as I got when using mozregression. However, I would disagree with it just affecting Print Preview, as it does appear on the physical printed page as well.
Flags: needinfo?(kyle)
Regressed by: 1308876
No longer regressed by: 1308876
No longer blocks: 1308876
Regressed by: 1308876
Component: Print Preview → Layout

This bug was fixed in this range, which means it was fixed by bug 1427141.

Assignee: nobody → mats
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox73: --- → fixed
Depends on: 1427141
Flags: in-testsuite?
Resolution: --- → FIXED
Target Milestone: --- → mozilla73

Do the tests landed in bug 1427141 cover this case or should we land the test from this bug still?

status-firefox71: --- → wontfix
status-firefox72: --- → wontfix
status-firefox-esr68: --- → wontfix
Flags: needinfo?(mats)

The tests for bug 1427141 is probably enough to cover this case. This bug doesn't really have a reduced testcase anyway.

Flags: needinfo?(mats)
Flags: in-testsuite? → in-testsuite-
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: