Skip to content

Commit e162c06

Browse files
emilionatechapin
authored andcommitted
Update cssparser to fix correctness issue with serialization.
This issue was also exposed via CSS variables, and I've added a test for that since it's how it was originally found in bug 1498188. See w3c/csswg-drafts#4088 and servo/rust-cssparser#251 Differential Revision: https://phabricator.services.mozilla.com/D37148 bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1563892 gecko-commit: d9f121b5943ecb4c917d54ea63824872bded0f38 gecko-integration-branch: central gecko-reviewers: heycam
1 parent 95f7831 commit e162c06

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

css/css-variables/variable-substitution-basic.html

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,12 @@
3838
expectedValue:"0px 0px",
3939
style:"--gap: 20;border-spacing: var(--gap)px;"
4040
},
41+
{
42+
testName:"You can't build up a single token where part of it is provided by a variable (percentages)",
43+
propertyName:"text-indent",
44+
expectedValue:"0px",
45+
style:"--v: 20;text-indent: var(--v)%;"
46+
},
4147
{
4248
testName:"Multiple variable references in a single property",
4349
propertyName:"border-spacing",
@@ -115,4 +121,4 @@
115121
});
116122
</script>
117123
</body>
118-
</html>
124+
</html>

0 commit comments

Comments
 (0)