Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(364)

Issues Search
    My Issues | Starred     Open | Closed | All

Side by Side Diff: LayoutTests/fast/css-grid-layout/min-width-height-auto.html

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
Patch Set: Removed stretching logic optimization, clearing override values by default. Created 5 years, 5 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:
View unified diff | Download patch
« LayoutTests/fast/css-grid-layout/grid-items-should-not-be-stretched-when-height-or-margin-change.html ('K') | « LayoutTests/fast/css-grid-layout/grid-items-should-not-be-stretched-when-height-or-margin-change-expected.txt ('k') | LayoutTests/fast/css-grid-layout/min-width-height-auto-expected.txt » ('j') | Source/core/layout/LayoutGrid.cpp » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 <html> 2 <html>
3 <link href="resources/grid.css" rel="stylesheet"> 3 <link href="resources/grid.css" rel="stylesheet">
4 <style> 4 <style>
5 .grid { 5 .grid {
6 grid-template-columns: 5px; 6 grid-template-columns: 5px;
7 grid-template-rows: 5px; 7 grid-template-rows: 5px;
8 } 8 }
9 9
10 .container { 10 .container {
(...skipping 17 matching lines...) Expand all
28 28
29 .maxSmaller { 29 .maxSmaller {
30 max-width: 10px; 30 max-width: 10px;
31 max-height: 10px; 31 max-height: 10px;
32 } 32 }
33 33
34 .maxBigger { 34 .maxBigger {
35 max-width: 150px; 35 max-width: 150px;
36 max-height: 75px; 36 max-height: 75px;
37 } 37 }
38
39 .justifyStart {
40 justify-items: start;
41 }
38 </style> 42 </style>
39 <script src="../../resources/check-layout.js"></script> 43 <script src="../../resources/check-layout.js"></script>
40 <body onload="checkLayout('.grid')"> 44 <body onload="checkLayout('.grid')">
41 45
42 <p>This test checks min-width|height auto behavior for grids</p> 46 <p>This test checks min-width|height auto behavior for grids</p>
43 47
44 <div class="container"> 48 <div class="container">
45 <div class="grid"> 49 <div class="grid">
46 <div class="ahem" data-expected-width="100" data-expected-height="25">XX XX</div> 50 <div class="ahem" data-expected-width="100" data-expected-height="25">XX XX</div>
47 </div> 51 </div>
(...skipping 16 matching lines...) Expand all
64 <div class="ahem maxSmaller" data-expected-width="10" data-expected-heig ht="10">XXXX</div> 68 <div class="ahem maxSmaller" data-expected-width="10" data-expected-heig ht="10">XXXX</div>
65 </div> 69 </div>
66 </div> 70 </div>
67 71
68 <div class="container"> 72 <div class="container">
69 <div class="grid"> 73 <div class="grid">
70 <div class="ahem maxBigger" data-expected-width="100" data-expected-heig ht="25">XXXX</div> 74 <div class="ahem maxBigger" data-expected-width="100" data-expected-heig ht="25">XXXX</div>
71 </div> 75 </div>
72 </div> 76 </div>
73 77
78 <div class="container">
79 <div class="grid justifyStart">
80 <div class="ahem" data-expected-width="100" data-expected-height="25">XX XX</div>
81 </div>
82 </div>
83
84 <div class="container">
85 <div class="grid justifyStart">
86 <div class="ahem minSmaller" data-expected-width="10" data-expected-heig ht="10">XXXX</div>
87 </div>
88 </div>
89
90 <div class="container">
91 <div class="grid justifyStart">
92 <div class="ahem minBigger" data-expected-width="150" data-expected-heig ht="75">XXXX</div>
93 </div>
94 </div>
95
96 <div class="container">
97 <div class="grid justifyStart">
98 <div class="ahem maxSmaller" data-expected-width="10" data-expected-heig ht="10">XXXX</div>
99 </div>
100 </div>
101
102 <div class="container">
103 <div class="grid justifyStart">
104 <div class="ahem maxBigger" data-expected-width="100" data-expected-heig ht="25">XXXX</div>
105 </div>
106 </div>
107
108 <div class="container">
109 <div class="grid">
110 <div class="ahem" data-expected-width="75" data-expected-height="50">XXX X</div>
111 </div>
112 </div>
Manuel Rego 2015/07/16 10:45:05 Nit: Maybe move this one before all the "justifySt
jfernandez 2015/07/16 11:06:03 Well, we are testing here a different case, which
113
114 <div class="container">
115 <div class="grid justifyStart">
116 <div class="ahem" data-expected-width="75" data-expected-height="50">XXX X</div>
117 </div>
118 </div>
119
74 </body> 120 </body>
75 </html> 121 </html>
OLDNEW
« LayoutTests/fast/css-grid-layout/grid-items-should-not-be-stretched-when-height-or-margin-change.html ('K') | « LayoutTests/fast/css-grid-layout/grid-items-should-not-be-stretched-when-height-or-margin-change-expected.txt ('k') | LayoutTests/fast/css-grid-layout/min-width-height-auto-expected.txt » ('j') | Source/core/layout/LayoutGrid.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698