Skip to content

Commit 936f5b8

Browse files
committed
Merge pull request #125 from nacin/media-feature-spaces
Allow spaces within media query features
2 parents f034400 + 88f40eb commit 936f5b8

File tree

2 files changed

+15
-1
lines changed

2 files changed

+15
-1
lines changed

src/css/Parser.js

+2
Original file line numberDiff line numberDiff line change
@@ -526,6 +526,8 @@ Parser.prototype = function(){
526526
*/
527527
var tokenStream = this._tokenStream;
528528

529+
this._readWhitespace();
530+
529531
tokenStream.mustMatch(Tokens.IDENT);
530532

531533
return SyntaxUnit.fromToken(tokenStream.token());

tests/css/Parser.js

+13-1
Original file line numberDiff line numberDiff line change
@@ -1498,7 +1498,7 @@
14981498
Assert.isTrue(true); //just don't want an error
14991499
},
15001500

1501-
testMediaWithViewport: function(){
1501+
testMediaWithViewport: function(){
15021502
var parser = new Parser({ strict: true});
15031503
var result = parser.parse("@media { @viewport {} }");
15041504
Assert.isTrue(true); //just don't want an error
@@ -1528,6 +1528,18 @@
15281528
Assert.isTrue(true); //just don't want an error
15291529
},
15301530

1531+
testMediaWithSimpleExpressionWithSpaces: function(){
1532+
var parser = new Parser({ strict: true});
1533+
var result = parser.parse("@media ( max-width:400px ) { }");
1534+
Assert.isTrue(true); //just don't want an error
1535+
},
1536+
1537+
testMediaWithComplexExpressionWithSpaces: function(){
1538+
var parser = new Parser({ strict: true});
1539+
var result = parser.parse("@media all and ( max-width:400px ) { }");
1540+
Assert.isTrue(true); //just don't want an error
1541+
},
1542+
15311543
testViewport: function(){
15321544
var parser = new Parser({ strict: true});
15331545
var result = parser.parse("@viewport { width: 397px; }");

0 commit comments

Comments
 (0)