third_party/WebKit/LayoutTests/fragmentation/single-line-cells-repeating-thead-starts-middle-of-page-break-after-avoid-2.html - Issue 2388613002: Move below headers even when row doesn't require a strut

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: third_party/WebKit/LayoutTests/fragmentation/single-line-cells-repeating-thead-starts-middle-of-page-break-after-avoid-2.html

Issue 2388613002: Move below headers even when row doesn't require a strut (Closed)

Patch Set: bug 642814 Created 4 years, 2 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

OLD	NEW
(Empty)
	1 <!DOCTYPE html>

	2 <style>

	3 table {

	4 font-size: 12pt;
	mstensho (USE GERRIT) 2016/10/04 21:02:38 Would 16px work too? Would 16px work too?
	5 }

	6 td, th {

	7 background-color: #ddd;

	8 border: 1px solid black;

	9 }

	10 thead, tr {

	11 break-inside: avoid;

	12 }

	13 </style>

	14 <p>crbug.com/624814: A table header group should not repeat on each page if we c an't fit at least one content row below it.</p>
	mstensho (USE GERRIT) 2016/10/04 21:02:38 That doesn't seem to be what this test is testing. That doesn't seem to be what this test is testing. It's about unwanted overlap in the second column, right?
	15 <div style="-webkit-columns:3; line-height: 18px; column-fill: auto; height:91px ; background-color: yellow;">
	mstensho (USE GERRIT) 2016/10/04 21:02:38 No need for -webkit- prefix. No need for -webkit- prefix.
	16 <br>

	17 <table>

	18 <thead>

	19 <tr>

	20 <th>Col 1</th>

	21 <th>Col 2</th>

	22 </tr>

	23 </thead>

	24 <tr><td>Te</td><td>xt</td></tr>

	25 <tr><td>Te</td><td>xt</td></tr>

	26 <tr><td>Te</td><td>xt</td></tr>

	27 </table>

	28 </div>

	29

OLD	NEW