diff options
| author | Emily McDonough <emcdonough@mozilla.com> | 2020-03-19 22:11:48 +0000 |
|---|---|---|
| committer | Emilio Cobos Álvarez <emilio@crisal.io> | 2020-04-16 16:35:07 +0200 |
| commit | e0b4619fa58c756cbde4fa5480f40c66abb6122b (patch) | |
| tree | f0a092e91cb57a6ea3f21235b68df3b5e974b06d /components/style | |
| parent | b85b6ac16fb7c88da9c893764ac3a9ebe9e8a581 (diff) | |
| download | servo-e0b4619fa58c756cbde4fa5480f40c66abb6122b.tar.gz servo-e0b4619fa58c756cbde4fa5480f40c66abb6122b.zip | |
style: Enable multiple grid repeat values in Servo.
Differential Revision: https://phabricator.services.mozilla.com/D60931
Diffstat (limited to 'components/style')
| -rw-r--r-- | components/style/values/generics/grid.rs | 8 | ||||
| -rw-r--r-- | components/style/values/specified/grid.rs | 10 |
2 files changed, 0 insertions, 18 deletions
diff --git a/components/style/values/generics/grid.rs b/components/style/values/generics/grid.rs index 1b95934e3c1..6f0f155912d 100644 --- a/components/style/values/generics/grid.rs +++ b/components/style/values/generics/grid.rs @@ -696,14 +696,6 @@ impl Parse for LineNameList { .take(num.value() as usize * names_list.len()), ), RepeatCount::AutoFill if fill_data.is_none() => { - // `repeat(autof-fill, ..)` should have just one line name. - // FIXME(bug 1341507) the above comment is wrong per: - // https://drafts.csswg.org/css-grid-2/#typedef-name-repeat - if names_list.len() != 1 { - return Err( - input.new_custom_error(StyleParseErrorKind::UnspecifiedError) - ); - } let fill_idx = line_names.len(); let fill_len = names_list.len(); fill_data = Some((fill_idx, fill_len)); diff --git a/components/style/values/specified/grid.rs b/components/style/values/specified/grid.rs index 4830aea24d6..54d31d5826d 100644 --- a/components/style/values/specified/grid.rs +++ b/components/style/values/specified/grid.rs @@ -185,16 +185,6 @@ impl TrackRepeat<LengthPercentage, Integer> { values.push(track_size); names.push(current_names); - if is_auto { - // FIXME: In the older version of the spec - // (https://www.w3.org/TR/2015/WD-css-grid-1-20150917/#typedef-auto-repeat), - // if the repeat type is `<auto-repeat>` we shouldn't try to parse more than - // one `TrackSize`. But in current version of the spec, this is deprecated - // but we are adding this for gecko parity. We should remove this when - // gecko implements new spec. - names.push(input.try(parse_line_names).unwrap_or_default()); - break; - } } else { if values.is_empty() { // expecting at least one <track-size> |