[Media Ranges] Fix a syntactic ambiguity

sass · Jul 25, 2018 · f81b1ec · f81b1ec
1 parent cf3283d
commit f81b1ec
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/accepted/media-ranges.changes.md b/accepted/media-ranges.changes.md
@@ -1,3 +1,8 @@
+## Draft 3
+
+* Disallow ambiguous binary operators in the `'(' Expression ')'` option for the
+  `MediaFeature` production.
+
 ## Draft 2
 
 * Refer to CSS's `<ident-token>` rather than a Sass-specific `Identifier`

diff --git a/accepted/media-ranges.md b/accepted/media-ranges.md
@@ -1,4 +1,4 @@
-# Range-Context Media Features: Draft 2
+# Range-Context Media Features: Draft 3
 
 *([Issue](https://github.com/sass/sass/issues/1864), [Changelog](media-ranges.changes.md))*
 
@@ -62,7 +62,7 @@ MediaQueryList ::= MediaQuery (',' MediaQuery)*
 MediaQuery     ::= MediaType | (MediaType 'and')? MediaFeature ('and' MediaFeature)*
 MediaType      ::= InterpolatedIdentifier InterpolatedIdentifier¹?
 MediaFeature   ::= Interpolation
-                 | '(' Expression ')'
+                 | '(' Expression² ')'
                  | '(' Expression² ':' Expression ')'
                  | '(' Expression² <mf-comparison> Expression² ')'
                  | '(' Expression² <mf-lt> Expression² <mf-lt> Expression² ')'