Generate CSSStyleDeclaration attributes from CSSProperties.json5 |
|||||||||
Issue descriptionIn https://github.com/w3c/csswg-drafts/issues/1089 I suggested changing the CSSOM spec to be more like Blink, without knowing exactly how that would work. Turns out that Gecko already does what the spec suggests, and that this results in a more reasonable behavior. Investigate if this would be a simple change in Blink, and if not revisit the spec issue. In particular, it would be worth measuring the increase in binary size to make sure it doesn't add 100s of KB. ⛆ |
|
|
,
Mar 12 2017
,
Dec 6 2017
,
Dec 16 2017
It'd be nice to get this fixed, there are WPT tests relying on the incorrect Blink behavior which I'm fixing in https://bugzilla.mozilla.org/show_bug.cgi?id=1425227.
,
May 31 2018
So I have a very rough WIP patch for this, but the main problem I ran into was for dashed attributes. Our current IDL parser doesn't allow dashes anywhere in an attribute name (contrary to the spec, which allows dashes anywhere other than the first character), meaning that attributes like font-size don't work. But even if that is fixed, WebIDL doesn't allow dashes as the first character of an identifier, meaning that -webkit/other prefixed properties cannot be attributes in WebIDL. Firefox avoids this issue as they have implemented a special extended attribute of [BindingAlias=] (see https://searchfox.org/mozilla-central/source/dom/bindings/GenerateCSS2PropertiesWebIDL.py#58) which means that they only define one attribute (the lower camel case one) for each property and the rest are aliased to that, without needing to be valid WebIDL identifiers.
,
Jun 7 2018
,
Jun 8 2018
|
||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by dstockwell@chromium.org
, Mar 12 2017Labels: -Type-Bug Type-Feature
Status: Available (was: Untriaged)