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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: LayoutTests/css3/flexbox/inline-flexbox-wrap-vertically-width-calculation.html

Issue 289903007: Inline flexbox width is wrongly calculated when wrapping vertically (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Addressing comments of patch set 6 Created 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | LayoutTests/css3/flexbox/inline-flexbox-wrap-vertically-width-calculation-expected.txt » ('j') | LayoutTests/css3/flexbox/multiline-justify-content.html » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/css3/flexbox/inline-flexbox-wrap-vertically-width-calculation.html
diff --git a/LayoutTests/css3/flexbox/inline-flexbox-wrap-vertically-width-calculation.html b/LayoutTests/css3/flexbox/inline-flexbox-wrap-vertically-width-calculation.html
new file mode 100644
index 0000000000000000000000000000000000000000..59723fada203bd06ca56837b266619813748dcb8
--- /dev/null
+++ b/LayoutTests/css3/flexbox/inline-flexbox-wrap-vertically-width-calculation.html
@@ -0,0 +1,91 @@
+<!DOCTYPE html>
+<html>
+<link href="resources/flexbox.css" rel="stylesheet">
+<style>
+.inline-flexbox {
+ background-color: #aaa;
+ position: relative;
+}
+
+.writing-mode-vertical {
+ -webkit-writing-mode: vertical-rl;
+}
+.inline-flexbox > * {
+ flex: none;
+}
+.inline-flexbox :nth-child(1) {
+ background-color: lightblue;
+}
+.inline-flexbox :nth-child(2) {
+ background-color: lightgreen;
+}
+.inline-flexbox :nth-child(3) {
+ background-color: pink;
+}
+.inline-flexbox :nth-child(4) {
+ background-color: yellow;
+}
+</style>
+<script src="../../resources/check-layout.js"></script>
+<body onload="checkLayout('.inline-flexbox');">
+<div class="inline-flexbox column align-content-flex-start wrap" data-expected-width="110" data-expected-height="60">
+ <div style="width: 100px; height: 20px" data-offset-x="0" data-offset-y="0"></div>
+ <div style="width: 50px; height: 10px" data-offset-x="0" data-offset-y="20"></div>
+ <div style="width: 70px; height: 10px" data-offset-x="0" data-offset-y="30"></div>
+ <div style="width: 110px; height: 20px" data-offset-x="0" data-offset-y="40"></div>
+</div>
+
+<div class="inline-flexbox column align-content-flex-start wrap" style="height: 35px" data-expected-width="80" data-expected-height="35">
+ <div style="width: 10px; height: 20px" data-offset-x="0" data-offset-y="0"></div>
+ <div style="width: 50px; height: 10px" data-offset-x="0" data-offset-y="20"></div>
+ <div style="width: 80px; height: 10px" data-offset-x="50" data-offset-y="0"></div>
+ <div style="width: 40px; height: 20px" data-offset-x="50" data-offset-y="10"></div>
+</div>
+
+<div style="width: 300px;">
+<div class="inline-flexbox column align-content-flex-start wrap" style="width: 50%" data-expected-width="150" data-expected-height="60">
tony 2014/07/22 16:42:02 I don't really understand the point of a test with
harpreet.sk 2014/07/23 13:43:24 This test case was added earlier to check for the
+ <div style="width: 100px; height: 20px" data-offset-x="0" data-offset-y="0"></div>
+ <div style="width: 10px; height: 10px" data-offset-x="0" data-offset-y="20"></div>
+ <div style="width: 50px; height: 10px" data-offset-x="0" data-offset-y="30"></div>
+ <div style="width: 100px; height: 20px" data-offset-x="0" data-offset-y="40"></div>
+</div>
+</div>
+
+<div style="width: 300px;">
+<div class="inline-flexbox column align-content-flex-start wrap" style="width: 50%; height: 35px;" data-expected-width="150" data-expected-height="35">
+ <div style="width: 100px; height: 20px" data-offset-x="0" data-offset-y="0"></div>
+ <div style="width: 10px; height: 10px" data-offset-x="0" data-offset-y="20"></div>
+ <div style="width: 50px; height: 10px" data-offset-x="100" data-offset-y="0"></div>
+ <div style="width: 100px; height: 20px" data-offset-x="100" data-offset-y="10"></div>
+</div>
+</div>
+
+<div class="inline-flexbox column align-content-flex-start wrap writing-mode-vertical" data-expected-width="280" data-expected-height="20">
+ <div style="width: 50px; height: 20px" data-offset-x="230" data-offset-y="0"></div>
tony 2014/07/22 16:42:02 I would add more variety to the heights in some of
harpreet.sk 2014/07/23 13:43:24 Done.
+ <div style="width: 100px; height: 10px" data-offset-x="130" data-offset-y="0"></div>
+ <div style="width: 30px; height: 10px" data-offset-x="100" data-offset-y="0"></div>
+ <div style="width: 100px; height: 20px" data-offset-x="0" data-offset-y="0"></div>
+</div>
+
+<div class="inline-flexbox column align-content-flex-start wrap writing-mode-vertical" style="width: 200px" data-expected-width="200" data-expected-height="20">
+ <div style="width: 50px; height: 20px" data-offset-x="150" data-offset-y="0"></div>
+ <div style="width: 100px; height: 10px" data-offset-x="50" data-offset-y="0"></div>
+ <div style="width: 30px; height: 10px" data-offset-x="20" data-offset-y="0"></div>
+ <div style="width: 100px; height: 20px" data-offset-x="100" data-offset-y="20"></div>
+</div>
+
+<div class="inline-flexbox column align-content-flex-start wrap writing-mode-vertical" data-expected-width="280" data-expected-height="20">
+ <div class="writing-mode-vertical" style="width: 100px; height: 20px" data-offset-x="180" data-offset-y="0"></div>
tony 2014/07/22 16:42:02 Why do you set the writing mode on both the flexbo
harpreet.sk 2014/07/23 13:43:24 Oops! sorry i forget that. This test case has been
+ <div class="writing-mode-vertical" style="width: 50px; height: 10px" data-offset-x="130" data-offset-y="0"></div>
+ <div class="writing-mode-vertical" style="width: 30px; height: 10px" data-offset-x="100" data-offset-y="0"></div>
+ <div class="writing-mode-vertical" style="width: 100px; height: 20px" data-offset-x="0" data-offset-y="0"></div>
+</div>
+
+<div class="inline-flexbox column align-content-flex-start wrap writing-mode-vertical" style="width: 200px" data-expected-width="200" data-expected-height="20">
+ <div class="writing-mode-vertical" style="width: 60px; height: 20px" data-offset-x="140" data-offset-y="0"></div>
+ <div class="writing-mode-vertical" style="width: 70px; height: 10px" data-offset-x="70" data-offset-y="0"></div>
+ <div class="writing-mode-vertical" style="width: 50px; height: 10px" data-offset-x="20" data-offset-y="0"></div>
+ <div class="writing-mode-vertical" style="width: 100px; height: 20px" data-offset-x="100" data-offset-y="20"></div>
+</div>
+</body>
+</html>
« no previous file with comments | « no previous file | LayoutTests/css3/flexbox/inline-flexbox-wrap-vertically-width-calculation-expected.txt » ('j') | LayoutTests/css3/flexbox/multiline-justify-content.html » ('J')

Powered by Google App Engine
This is Rietveld 408576698