-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[wpt] Fix grid-item (no) aspect-ratio tests.
Renaming scheme got lost, however basically: grid-item-no-aspect-ratio-stretch-4.html -> grid-item-aspect-ratio-stretch-1.html grid-item-no-aspect-ratio-stretch-5.html -> grid-item-aspect-ratio-stretch-2.html grid-item-no-aspect-ratio-stretch-6.html -> grid-item-aspect-ratio-stretch-3.html grid-item-no-aspect-ratio-stretch-7.html -> grid-item-aspect-ratio-stretch-4.html These tests all had a viewBox defining a valid aspect-ratio. Due to: w3c/csswg-drafts#6286 (comment) These tests *should* have an aspect-ratio, and when stretched in one dimension, should reflect to the other dimension (if unconstrained). See: w3c/csswg-drafts#5713 (comment) The below two tests basically just got renamed: grid-item-no-aspect-ratio-stretch-8.html -> grid-item-no-aspect-ratio-stretch-4.html grid-item-no-aspect-ratio-stretch-9.html -> grid-item-no-aspect-ratio-stretch-5.html grid-item-no-aspect-ratio-stretch-10.html -> grid-item-no-aspect-ratio-stretch-6.html But tests updated to correctly assert that the natural size would still be respected. To all these test-cases I also added "grid-template: 100% / 100%;" as there is further complexity when inside an auto row/column which is tested elsewhere. (Transferred minimum size for replaced elements with an aspect-ratio). Bug: 1114013 Change-Id: I062f67e291cc62fa63a53370595780dae16abf3b
- Loading branch information
1 parent
396d582
commit e4aa377
Showing
15 changed files
with
201 additions
and
67 deletions.
There are no files selected for viewing
51 changes: 51 additions & 0 deletions
51
css/css-grid/alignment/grid-item-aspect-ratio-stretch-1-ref.html
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,51 @@ | ||
| <!doctype html> | ||
| <meta charset="utf-8"> | ||
| <title>Reference: stretching works for replaced items with a fallback aspect ratio</title> | ||
| <link rel="author" title="Mats Palmgren" href="mailto:mats@mozilla.com"> | ||
| <link rel="author" title="Mozilla" href="https://mozilla.org"> | ||
| <style> | ||
| body { | ||
| line-height: 0; | ||
| } | ||
|
|
||
| div { | ||
| display: inline-block; | ||
| height: 250px; | ||
| width: 350px; | ||
| background: grey; | ||
| margin: 10px; | ||
| vertical-align: top; | ||
| } | ||
|
|
||
| img { | ||
| display: block; | ||
| } | ||
|
|
||
| .justify { | ||
| width: 350px; | ||
| } | ||
| .align { | ||
| height: 250px; | ||
| } | ||
| </style> | ||
| <div> | ||
| <img class="align justify"> | ||
| </div> | ||
| <div> | ||
| <img class="align"> | ||
| </div> | ||
| <div> | ||
| <img class="justify"> | ||
| </div> | ||
| <div> | ||
| <img style="width:10px; height:20px"> | ||
| </div> | ||
|
|
||
| <script> | ||
| var url = 'data:image/svg+xml,<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 50 100"><circle cx="50%" cy="50%" r="50%" fill="blue"/></svg>' | ||
| var imgs = document.querySelectorAll('img'); | ||
| for (var i = 0; i < imgs.length; ++i) { | ||
| var img = imgs[i]; | ||
| img.src = url; | ||
| } | ||
| </script> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
51 changes: 51 additions & 0 deletions
51
css/css-grid/alignment/grid-item-aspect-ratio-stretch-2-ref.html
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,51 @@ | ||
| <!doctype html> | ||
| <meta charset="utf-8"> | ||
| <title>Reference: stretching works for replaced items with a fallback aspect ratio</title> | ||
| <link rel="author" title="Mats Palmgren" href="mailto:mats@mozilla.com"> | ||
| <link rel="author" title="Mozilla" href="https://mozilla.org"> | ||
| <style> | ||
| body { | ||
| line-height: 0; | ||
| } | ||
|
|
||
| div { | ||
| display: inline-block; | ||
| height: 250px; | ||
| width: 350px; | ||
| background: grey; | ||
| margin: 10px; | ||
| vertical-align: top; | ||
| } | ||
|
|
||
| img { | ||
| display: block; | ||
| } | ||
|
|
||
| .justify { | ||
| width: 350px; | ||
| } | ||
| .align { | ||
| height: 250px; | ||
| } | ||
| </style> | ||
| <div> | ||
| <img class="align justify"> | ||
| </div> | ||
| <div> | ||
| <img class="align"> | ||
| </div> | ||
| <div> | ||
| <img class="justify"> | ||
| </div> | ||
| <div> | ||
| <img style="width:20px; height:40px"> | ||
| </div> | ||
|
|
||
| <script> | ||
| var url = 'data:image/svg+xml,<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 50 100"><circle cx="50%" cy="50%" r="50%" fill="blue"/></svg>' | ||
| var imgs = document.querySelectorAll('img'); | ||
| for (var i = 0; i < imgs.length; ++i) { | ||
| var img = imgs[i]; | ||
| img.src = url; | ||
| } | ||
| </script> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
52 changes: 52 additions & 0 deletions
52
css/css-grid/alignment/grid-item-aspect-ratio-stretch-3-ref.html
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,52 @@ | ||
| <!doctype html> | ||
| <meta charset="utf-8"> | ||
| <title>Reference: stretching works for replaced items with a fallback aspect ratio</title> | ||
| <link rel="author" title="Mats Palmgren" href="mailto:mats@mozilla.com"> | ||
| <link rel="author" title="Mozilla" href="https://mozilla.org"> | ||
| <style> | ||
| body { | ||
| line-height: 0; | ||
| } | ||
|
|
||
| div { | ||
| display: inline-block; | ||
| height: 250px; | ||
| width: 350px; | ||
| background: grey; | ||
| margin: 10px; | ||
| vertical-align: top; | ||
| } | ||
|
|
||
| img { | ||
| display: block; | ||
| background: blue; | ||
| } | ||
|
|
||
| .justify { | ||
| width: 350px; | ||
| } | ||
| .align { | ||
| height: 250px; | ||
| } | ||
| </style> | ||
| <div> | ||
| <img class="align justify"> | ||
| </div> | ||
| <div> | ||
| <img class="align"> | ||
| </div> | ||
| <div> | ||
| <img class="justify"> | ||
| </div> | ||
| <div> | ||
| <img style="width:0px; height:20px"> | ||
| </div> | ||
|
|
||
| <script> | ||
| var url = 'data:image/svg+xml,<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 50 100"></svg>' | ||
| var imgs = document.querySelectorAll('img'); | ||
| for (var i = 0; i < imgs.length; ++i) { | ||
| var img = imgs[i]; | ||
| img.src = url; | ||
| } | ||
| </script> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
6 changes: 4 additions & 2 deletions
6
.../grid-item-no-aspect-ratio-stretch-7.html → ...ent/grid-item-aspect-ratio-stretch-4.html
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.