RE: [css3-images] 2011/12/01 ED section 4.2 review notes

Brad:
> What was pasted into the irc for the telecon was a simplified version
> that hid nuances that I clearly the four of us hadn't quite reached
> consensus on. I think you jumped the gun on it. The WG would likely
> quickly resolve on a refinement if the four of us could agree. It's a
> cop out to refuse to do a little better along the same lines now
> because of the general WG resolution.

This was my part of my expressed concern in my first mail ([css3-images] radial gradient syntax ED changed and immediately  WD'd?) after the TelCon.

The four of us put in time to converge on a new proposal.

With two of us not involved, the syntax was changed and it was presented that there was still consensus -- which was not correct.  The "new new" grammar hadn't even been discussed/reviewed much less agreed upon.



Regarding solving the issues...

ED 2011/12/06:

<radial-gradient> = radial-gradient(
  [ [ circle               || <length> ]                          [ at <position> ]? , | 
    [ ellipse              || [ <length> | <percentage> ]{2} ]    [ at <position> ]? , |
    [ [ circle | ellipse ] || <extent-keyword> ]                  [ at <position> ]? , |
    at <position> ,
  ]?
  <color-stop> [ , <color-stop> ]+
)
<extent-keyword> = closest-corner | closest-side | farthest-corner | farthest-side



Grammar simplification proposal A:

<radial-gradient> = radial-gradient(
  [ <length>                               [ at <position> ]? , | 
    [ <length> | <percentage> ]{2}         [ at <position> ]? , |
    <extent-keyword> [ circle | ellipse ]? [ at <position> ]? , |
    at <position> ,
  ]?
  <color-stop> [ , <color-stop> ]+
)
<extent-keyword> = closest-corner | closest-side | farthest-corner | farthest-side


A solves the following problems:
1. redundant "circle" no longer allowed when single length is provided for size
2. redundant "ellipse" no longer allowed when length/percentage pair is provided for size
3. remove gratuitous reordering capability from the only place that (now) remains in the grammar
4. put circle/ellipse keyword after extent-keyword (because both Brad and Elika agree it makes more sense in English, which is the language currently used -- see "at")



Grammar simplification proposal B:

<radial-gradient> = radial-gradient(
  [ <length>                       [ at <position> ]? , | 
    [ <length> | <percentage> ]{2} [ at <position> ]? , |
    <extent-keyword> [circle]?     [ at <position> ]? , |
    at <position> ,
  ]?
  <color-stop> [ , <color-stop> ]+
)
<extent-keyword> = closest-corner | closest-side | farthest-corner | farthest-side


B solves the following problems:
1-4.
5. keyword ellipse no longer needed



I prefer both A and B to the WD 12/06 grammar.

Received on Wednesday, 7 December 2011 18:09:29 UTC