[css-borders-4][editorial] Rewrite syntaxes with property value range productions #12736
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
It might not be obvious why property value range productions should be preferred (imo), so taking
corner-*
shorthands as an example, its current syntax is:When replaced with
<'border-top-left-radius'> || <'corner-top-left-shape'>
:corner-*-shape
is later updated to eg.<corner-shape-value> | none
<length-percentage>
value should be when omittedApplying this change left the extended definition of
<corner-shape-value>
at an inappropriate place, below the definition table ofcorner-shape
, while it was no longer used in its syntax.So I took changes a little further and ended with moving/rewriting corner shaping sections to make all *Corners" sections more consistent, in a separate commit.
I also made an unrelated editorial change. Although experienced CSS authors might intuitevily guess this, I suggest to explicitly defining that
corner-shape
values are expanded clockwise starting from the top corner, and what their default values when omitted. I preferred to suggest this change in a single PR to avoid merge conflicts.