[css-multicol] style: Implement percentages for 'column-gap'.

Bug: 1398537 Reviewed-by: emilio,dholbert
servo · Apr 1, 2018 · 3532f64 · 3532f64
1 parent 9677d5c
commit 3532f64
Show file tree

Hide file tree

Showing 5 changed files with 26 additions and 12 deletions.
diff --git a/components/layout/multicol.rs b/components/layout/multicol.rs
@@ -106,6 +106,7 @@ impl Flow for MulticolFlow {
         {
             let column_style = self.block_flow.fragment.style.get_column();
 
+            // FIXME(#20498): This should support percentages too.
             let column_gap = match column_style.column_gap {
                 Either::First(len) => len.into(),
                 Either::Second(_normal) => self.block_flow.fragment.style.get_font().font_size.size(),

diff --git a/components/style/properties/gecko.mako.rs b/components/style/properties/gecko.mako.rs
@@ -1445,7 +1445,7 @@ impl Clone for ${style_struct.gecko_struct_name} {
         "length::LengthOrAuto": impl_style_coord,
         "length::LengthOrNormal": impl_style_coord,
         "length::NonNegativeLengthOrAuto": impl_style_coord,
-        "length::NonNegativeLengthOrNormal": impl_style_coord,
+        "length::NonNegativeLengthOrPercentageOrNormal": impl_style_coord,
         "Length": impl_absolute_length,
         "LengthOrNormal": impl_style_coord,
         "LengthOrPercentage": impl_style_coord,

diff --git a/components/style/properties/longhand/column.mako.rs b/components/style/properties/longhand/column.mako.rs
@@ -29,14 +29,21 @@ ${helpers.predefined_type(
     servo_restyle_damage="rebuild_and_reflow",
 )}
 
-${helpers.predefined_type("column-gap",
-                          "length::NonNegativeLengthOrNormal",
-                          "Either::Second(Normal)",
-                          extra_prefixes="moz",
-                          servo_pref="layout.columns.enabled",
-                          animation_value_type="NonNegativeLengthOrNormal",
-                          spec="https://drafts.csswg.org/css-multicol/#propdef-column-gap",
-                          servo_restyle_damage = "reflow")}
+
+<%
+# FIXME(#20498): Servo should support percentages in column-gap.
+col_gap_type = "NonNegativeLengthOrPercentageOrNormal" if product == "gecko" else "NonNegativeLengthOrNormal"
+%>
+${helpers.predefined_type(
+    "column-gap",
+    "length::%s" % col_gap_type,
+    "Either::Second(Normal)",
+    extra_prefixes="moz",
+    servo_pref="layout.columns.enabled",
+    animation_value_type=col_gap_type,
+    spec="https://drafts.csswg.org/css-multicol/#propdef-column-gap",
+    servo_restyle_damage = "reflow",
+)}
 
 ${helpers.single_keyword("column-fill", "balance auto", extra_prefixes="moz",
                          products="gecko", animation_value_type="discrete",

diff --git a/components/style/values/computed/length.rs b/components/style/values/computed/length.rs
@@ -897,6 +897,9 @@ pub type NonNegativeLengthOrAuto = Either<NonNegativeLength, Auto>;
 /// Either a computed NonNegativeLength or the `normal` keyword.
 pub type NonNegativeLengthOrNormal = Either<NonNegativeLength, Normal>;
 
+/// Either a computed NonNegativeLengthOrPercentage or the `normal` keyword.
+pub type NonNegativeLengthOrPercentageOrNormal = Either<NonNegativeLengthOrPercentage, Normal>;
+
 /// A type for possible values for min- and max- flavors of width, height,
 /// block-size, and inline-size.
 #[allow(missing_docs)]

diff --git a/components/style/values/specified/length.rs b/components/style/values/specified/length.rs
@@ -679,9 +679,6 @@ impl NonNegativeLength {
     }
 }
 
-/// Either a NonNegativeLength or the `normal` keyword.
-pub type NonNegativeLengthOrNormal = Either<NonNegativeLength, Normal>;
-
 /// Either a NonNegativeLength or the `auto` keyword.
 pub type NonNegativeLengthOrAuto = Either<NonNegativeLength, Auto>;
 
@@ -1020,6 +1017,12 @@ impl Parse for LengthOrPercentageOrNone {
 /// A wrapper of LengthOrPercentage, whose value must be >= 0.
 pub type NonNegativeLengthOrPercentage = NonNegative<LengthOrPercentage>;
 
+/// Either a computed NonNegativeLength or the `normal` keyword.
+pub type NonNegativeLengthOrNormal = Either<NonNegativeLength, Normal>;
+
+/// Either a NonNegativeLengthOrPercentage or the `normal` keyword.
+pub type NonNegativeLengthOrPercentageOrNormal = Either<NonNegativeLengthOrPercentage, Normal>;
+
 impl From<NoCalcLength> for NonNegativeLengthOrPercentage {
     #[inline]
     fn from(len: NoCalcLength) -> Self {