Chromium Code Reviews Chromium Code Reviews
Issue 1228983003:
[CSS Grid Layout] Do not stretch always grid items with auto width (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: LayoutTests/fast/css-grid-layout/relayout-align-items-changed.html |
| diff --git a/LayoutTests/fast/css-grid-layout/relayout-align-items-changed.html b/LayoutTests/fast/css-grid-layout/relayout-align-items-changed.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..c3e82cf64ab39ca12fd0d03e923c1979bae6c0ba |
| --- /dev/null |
| +++ b/LayoutTests/fast/css-grid-layout/relayout-align-items-changed.html |
| @@ -0,0 +1,37 @@ |
| +<!DOCTYPE HTML> |
| +<link href="resources/grid.css" rel="stylesheet"> |
| +<style> |
| +.grid { |
| + grid: 100px 100px / 150px; |
| + width: 200px; |
|
svillar
2015/07/10 14:00:52
Do we need width: here?
jfernandez
2015/07/10 15:07:24
It's just to make grid container fit grid tracks s
|
| + position: relative; |
| +} |
| +#fromStretch { align-items: stretch; } |
| +#toStretch { align-items: end; } |
| +</style> |
| +<script src="../../resources/check-layout.js"></script> |
| +<p style="height: 20px">Tests how a align-items style change requires a relayout of the grid and previously stretched items.</p> |
| +<p>The grid bellow had initially 'align-items: end' and was changed to 'stretch'.</p> |
|
svillar
2015/07/10 14:00:52
Why height on <p> element?
jfernandez
2015/07/10 15:07:24
A mistake, I will remove it.
|
| +<div id="toStretch" class="grid"> |
| + <div style="width: 50px;" class="firstRowFirstColumn" data-expected-height="150" data-offset-y="0"> |
|
svillar
2015/07/10 14:00:52
Can we set width on its child the same we do for h
jfernandez
2015/07/10 15:07:24
It won't affect the test, definitively, because we
|
| + <div style="height: 50px"></div> |
| + </div> |
| + <div style="width: 50px;" class="firstRowSecondColumn" data-expected-height="150" data-offset-y="0"> |
| + <div style="height: 100px"></div> |
| + </div> |
| +</div> |
| +<p>The grid bellow was initially stretched and it has now 'align-items: center'.</p> |
| +<div id="fromStretch" class="grid"> |
| + <div style="width: 50px;" class="firstRowFirstColumn" data-expected-height="50" data-offset-y="50"> |
| + <div style="height: 50px"></div> |
| + </div> |
| + <div style="width: 50px;" class="firstRowSecondColumn" data-expected-height="100" data-offset-y="25"> |
| + <div style="height: 100px"></div> |
| + </div> |
| +</div> |
| +<script> |
| +document.body.offsetLeft; |
| +document.getElementById("toStretch").style.alignItems = "stretch"; |
| +document.getElementById("fromStretch").style.alignItems = "center"; |
| +checkLayout(".grid"); |
| +</script> |
