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

Unified Diff: LayoutTests/fast/multicol/min-height-much-greater-than-content.html

Issue 1322513003: Multicol: Min-height should have no effect if content is taller and height is auto. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Use block instead of inline in test, so that we don't have to make assumptions about internal leadiā€¦ Created 5 years, 3 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
Index: LayoutTests/fast/multicol/min-height-much-greater-than-content.html
diff --git a/LayoutTests/fast/multicol/min-height-much-greater-than-content.html b/LayoutTests/fast/multicol/min-height-much-greater-than-content.html
new file mode 100644
index 0000000000000000000000000000000000000000..bccf8431f612448570d598d917325f9d58654e2c
--- /dev/null
+++ b/LayoutTests/fast/multicol/min-height-much-greater-than-content.html
@@ -0,0 +1,15 @@
+<!DOCTYPE html>
+<script src="../../resources/check-layout.js"></script>
+<p>Min-height should make the multicol container taller if min-height is greater than the content
+ height, but it should not affect column balancing. We specify column-fill:auto here, but the
+ columns should still be balanced, because height is auto.</p>
+<p>Below there should be a yellow square with the word "OKAY" inside - from left to right, with large letter spacing.</p>
+<div id="test" style="position:relative; -webkit-columns:8; -webkit-column-gap:0; column-fill:auto; width:160px; min-height:160px; line-height:20px; background:yellow;">
+ <div data-offset-y="0">O</div>
+ <div data-offset-y="0">K</div>
+ <div data-offset-y="0">A</div>
+ <div data-offset-y="0">Y</div>
+</div>
+<script>
+ checkLayout("#test");
+</script>

Powered by Google App Engine
This is Rietveld 408576698