-
Notifications
You must be signed in to change notification settings - Fork 676
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
[css-shapes][css3-backgrounds] No explicit <position> serialization steps in background spec #368
Comments
I don't think we want to change that behavior. It seems to me functions usually have special serialization rules, but not general property values. |
Does this mean we can serialize any way we want provided that it parses to the same thing? |
I don't think so. For one thing it makes writing web-platform-tests hard (not to mention making life hard for scripts that want to parse the result of |
It doesn't seem to me serialization of general specified value is defined anywhere... If we are trying to define it, it should match what engines are currently doing. My guess is that engines general just put the parsed tokens in the grammar order, so |
This should probably be defined in css-values spec I suppose. |
There's a fair bit of serialization defined in CSSOM, e.g. serializing a CSS value. But I don't think it covers |
I was looking for issues related to the serialization of
I guess that always uses the 2- or 4-value form comes from an old version of CSS Shape, but why this rule is also applied to
|
@cdoublev (and others) take a look at the discussion in #2274 for some of the things that led us to decide to only serialize to 2- and 4-value forms. The text in css-shapes was also supposed to also go in css-backgrounds-3 (I remember having plans to remove as much as I could from css-shapes and just refer to bg-position serialization when I could) |
Thanks, I would have found this issue if I had searched on this topic without the If I understood correctly :
|
This is now edited into css-values-4, see 0742ad6 Marking Needs Edits for CSS Shapes |
The issue seems also about missing serialization steps for |
https://drafts.csswg.org/css-shapes/#basic-shape-serialization
https://drafts.csswg.org/css-backgrounds-3/#position
The basic-shape spec mentions:
The backgrounds spec does not have serialization steps mentioned at all. Can we assume the same serialization steps?
(This doesn't match existing behavior -- browsers serialize
center center
ascenter center
instead of50% 50%
for background position)The text was updated successfully, but these errors were encountered: