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

Unified Diff: LayoutTests/fast/multicol/min-height-greater-than-height.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-greater-than-height.html
diff --git a/LayoutTests/fast/multicol/min-height-greater-than-height.html b/LayoutTests/fast/multicol/min-height-greater-than-height.html
new file mode 100644
index 0000000000000000000000000000000000000000..88e8b3e882c743a6e0f9a53cfe3cd3498b92e1ac
--- /dev/null
+++ b/LayoutTests/fast/multicol/min-height-greater-than-height.html
@@ -0,0 +1,19 @@
+<!DOCTYPE html>
+<script src="../../resources/check-layout.js"></script>
+<style>
+ .box { height:40px; background:blue; }
+</style>
+<p>If min-height is larger than height, the final height will be set according to
+ min-height. Content should fill columns to their full height (the same as the height of the
+ multicol container), since we request column-fill:auto, and height is non-auto, so we won't
+ balance.</p>
+<p>Below there should be a blue square to the left of a hotpink square.</p>
+<div id="test" style="position:relative; -webkit-columns:4; -webkit-column-gap:0; column-fill:auto; width:160px; height:40px; min-height:80px; background:hotpink;">
+ <div data-offset-y="0" class="box"></div>
+ <div data-offset-y="40" class="box"></div>
+ <div data-offset-y="0" class="box"></div>
+ <div data-offset-y="40" class="box"></div>
+</div>
+<script>
+ checkLayout("#test");
+</script>

Powered by Google App Engine
This is Rietveld 408576698