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-justify-items-changed.html |
| diff --git a/LayoutTests/fast/css-grid-layout/relayout-justify-items-changed.html b/LayoutTests/fast/css-grid-layout/relayout-justify-items-changed.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..7e195f3ddfdb0f29e628e488d78198e134af2473 |
| --- /dev/null |
| +++ b/LayoutTests/fast/css-grid-layout/relayout-justify-items-changed.html |
| @@ -0,0 +1,37 @@ |
| +<!DOCTYPE HTML> |
| +<link href="resources/grid.css" rel="stylesheet"> |
| +<style> |
| +.grid { |
| + grid: 150px / 100px 100px; |
| + width: 150px; |
|
svillar
2015/07/10 14:00:52
Do we really need width: here?
jfernandez
2015/07/10 15:07:25
See my comments above.
|
| + position: relative; |
| +} |
| +#fromStretch { justify-items: stretch; } |
| +#toStretch { justify-items: end; } |
| +</style> |
| +<script src="../../resources/check-layout.js"></script> |
| +<p style="height: 20px">Tests how a justify-items style change requires a relayout of the grid and previously stretched items.</p> |
|
svillar
2015/07/10 14:00:53
Why height on the <p> element?
jfernandez
2015/07/10 15:07:25
Done.
|
| +<p>The grid bellow had initially 'justify-items: end' and was changed to 'stretch'.</p> |
| +<div id="toStretch" class="grid"> |
| + <div style="height: 50px;" class="firstRowFirstColumn" data-expected-width="150" data-offset-x="0"> |
|
svillar
2015/07/10 14:00:52
Can we set the height on its child the same we do
jfernandez
2015/07/10 15:07:25
See my comments above.
|
| + <div style="width: 50px;"></div> |
| + </div> |
| + <div style="height: 50px;" class="secondRowFirstColumn" data-expected-width="150" data-offset-x="0"> |
| + <div style="width: 100px;"></div> |
| + </div> |
| +</div> |
| +<p>The grid bellow was initially stretched and it has now 'justify-items: center'.</p> |
| +<div id="fromStretch" class="grid"> |
| + <div style="height: 50px;" class="firstRowFirstColumn" data-expected-width="50" data-offset-x="50"> |
| + <div style="width: 50px;"></div> |
| + </div> |
| + <div style="height: 50px;" class="secondRowFirstColumn" data-expected-width="100" data-offset-x="25"> |
| + <div style="width: 100px;"></div> |
| + </div> |
| +</div> |
| +<script> |
| +document.body.offsetLeft; |
| +document.getElementById("toStretch").style.justifyItems = "stretch"; |
| +document.getElementById("fromStretch").style.justifyItems = "center"; |
| +checkLayout(".grid"); |
| +</script> |
