Closed Bug 1809490 Opened 1 year ago Closed 1 year ago

[wpt-sync] Sync PR 37850 - [table-fragmentation] Remove rowspan avoidance logic.

Categories

(Core :: Layout, task, P4)

task

Tracking

()

RESOLVED FIXED
111 Branch
Tracking Status
firefox111 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(Depends on 1 open bug, Blocks 1 open bug, )

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 37850 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/37850
Details from upstream follow.

Ian Kilpatrick <ikilpatrick@chromium.org> wrote:

[table-fragmentation] Remove rowspan avoidance logic.

When a cell was rowspanned we'd try to avoid breaking inside it as
this makes the logic circular. This however had the undesired effect
of rowspanned cells which are larger than the current page looking
undesirable, and creating large gaps when printing.

This removes the avoidance logic, at the expense of the rowspanned
cell not having the correct height if the table has grown due to
fragmentation.

Bug: 1396218
Change-Id: I10c0afc7bbc5d3ed3323b33417ee2ee22d191404
Reviewed-on: https://chromium-review.googlesource.com/4136462
WPT-Export-Revision: 4a5ca6333dd3a7ee2827a0789bad15fa1ceadce2

Component: web-platform-tests → Layout
Product: Testing → Core

CI Results

Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests

Status Summary

Firefox

FAIL: 1

Chrome

FAIL: 1

Safari

PASS: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/de2705e8ad68
[wpt PR 37850] - [table-fragmentation] Remove rowspan avoidance logic., a=testonly
https://hg.mozilla.org/integration/autoland/rev/0863eaa1e54c
[wpt PR 37850] - Update wpt metadata, a=testonly
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 111 Branch
You need to log in before you can comment on or make changes to this bug.