Re: [csswg-drafts] [css-values] Switch [..]! to ?? combinator

> Also possible, without needing a new combinator, is
> 
> ```[ <offset-position> | [ <offset-position>? <offset-path> [<offset-distance> || <offset-rotation>]? ] ] [ / <offset-anchor> ]?```

That seems wrong, because the right side of the pipe may be omitted completely. The correct syntax would then be (formatted for better readability):

    [
      <offset-position> |
      <offset-path> [ <offset-distance> || <offset-rotation> ]? |
      <offset-position> <offset-path> [ <offset-distance> || <offset-rotation> ]?
    ] [ / <offset-anchor> ]?

This makes the definition a bit long, so I can understand that @fantasai wants to add a new combinator. If one is to be created, I think it would look better like this, though:

    [ <offset-position> ?! [ <offset-path> [ <offset-distance> || <offset-rotation> ]? ] ]
    [ / <offset-anchor> ]?

or to use `?!` as a multiplier:

    [ <offset-position>?! [ <offset-path> [ <offset-distance> || <offset-rotation> ]? ]?! ]
    [ / <offset-anchor> ]?

Sebastian

-- 
GitHub Notification of comment by SebastianZ
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/687#issuecomment-291053271 using your GitHub account

Received on Monday, 3 April 2017 05:42:52 UTC