New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 675453 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Use other robhogan account instead.
Closed: Jun 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

Content of cell should avoid repeating headers when it straddles multiple pages

Project Member Reported by robhogan@chromium.org, Dec 18 2016

Issue description

This matches FF but not necessarily the spec: https://drafts.csswg.org/css-tables-3/#repeated-headers

FF's behaviour seems the most sensible.
 
single-line-cells-repeating-thead-cell-straddles-page.html
1.2 KB View Download
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 20 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0ab5bebc6d97953cf7ee417a80cf453a5145dd8e

commit 0ab5bebc6d97953cf7ee417a80cf453a5145dd8e
Author: robhogan <robhogan@gmail.com>
Date: Tue Dec 20 23:00:48 2016

Content of cell should avoid repeating headers when it straddles multiple pages

The right thing to do here is not covered by the spec, so for now just fix
our rendering bug by doing what firefox does.

Opened a discussion at https://lists.w3.org/Archives/Public/www-style/2016Dec/0070.html.

BUG= 675453 

Review-Url: https://codereview.chromium.org/2587673003
Cr-Commit-Position: refs/heads/master@{#439906}

[add] https://crrev.com/0ab5bebc6d97953cf7ee417a80cf453a5145dd8e/third_party/WebKit/LayoutTests/fragmentation/single-line-cells-repeating-thead-cell-straddles-page-expected.html
[add] https://crrev.com/0ab5bebc6d97953cf7ee417a80cf453a5145dd8e/third_party/WebKit/LayoutTests/fragmentation/single-line-cells-repeating-thead-cell-straddles-page.html
[modify] https://crrev.com/0ab5bebc6d97953cf7ee417a80cf453a5145dd8e/third_party/WebKit/Source/core/layout/LayoutBox.cpp
[modify] https://crrev.com/0ab5bebc6d97953cf7ee417a80cf453a5145dd8e/third_party/WebKit/Source/core/layout/LayoutTableRow.cpp
[modify] https://crrev.com/0ab5bebc6d97953cf7ee417a80cf453a5145dd8e/third_party/WebKit/Source/core/layout/LayoutTableRow.h
[modify] https://crrev.com/0ab5bebc6d97953cf7ee417a80cf453a5145dd8e/third_party/WebKit/Source/core/layout/LayoutTableSection.cpp

Status: Fixed (was: Started)

Sign in to add a comment