-
Notifications
You must be signed in to change notification settings - Fork 708
[css-align] Serialization of the alignment properties. #2229
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
There's a similar issue with |
Also cc @MatsPalmgren, who also had an opinion on this IIRC. |
Umm, I understand your point about first baseline and I'm working now on matching FF on this regard. I haven't considered the case of tough. Is it the same case ? cc @fantasai to gather more opinions. I think this is not the only CSSOM related issue we have, and I'm not sure we have a clear agreement about all these issues (see #1041 or #1033 for example) . I wouldn't mind to work on them as soon as we reach an agreement. |
CSS values should always be serialized to their shortest form. (It's a bug in the CSSOM spec that it no longer explicitly spells that out.) Regarding
|
What @MatsPalmgren said. Closing in favor of #1564, please re-open if there's some other consideration specific to css-align. |
It's worth noting though that |
Should properties serialize the default values? Right now Blink and WebKit do so, which I think it's a bug.
In particular:
Serializes in Blink as
unsafe start
, and onlystart
in FF. I think the computed value should serialize to the shortest form possible.This is annoying because it causes a lot of WPT failures on Gecko.
The text was updated successfully, but these errors were encountered: