[csswg-drafts] [css-shapes-2] The new definition of '<basic-shape>' should include 'path()' (#4270)

BorisChiou has just created a new issue for https://github.com/w3c/csswg-drafts:

== [css-shapes-2] The new definition of '<basic-shape>' should include 'path()' ==
The [definition of `<basic-shape>`](https://drafts.csswg.org/css-shapes-2/#typedef-basic-shape) for `shape-inside` in [css-shapes-2] doesn't include `path()` now. We define [`path()`](https://drafts.csswg.org/css-shapes-2/#funcdef-path) as a delta of supported basic shapes in [css-shapes-2], and it seems no other places mention it.
IIRC, at least four properties use `<basic-shape>`:
1. [`shape-outside`](https://drafts.csswg.org/css-shapes/#shape-outside-property): The [link of `<basic-shape>` in its value definition](https://drafts.csswg.org/css-shapes/#typedef-basic-shape) uses the definition of `<basic-shape>` in [css-shapes]
2. [`shape-inside`](https://drafts.csswg.org/css-shapes-2/#shape-inside-property): The [link of `<basic-shape>` in its value definition](https://drafts.csswg.org/css-shapes-2/#typedef-basic-shape) links to its own definition of `<basic-shape>`, in [css-shapes-2].
3. [`clip-path`](https://drafts.fxtf.org/css-masking-1/#the-clip-path): The same as that of `shape-inside`.
4. [`offset-path`](https://drafts.fxtf.org/motion-1/#offset-path-property): The same as `clip-path` and `shape-inside`. However, it also defines a different syntax of `path()` because `fill-rule` doesn't make sense on `offset-path`.

IMO, the link of the value definition of `<basic-shape>` for `shape-outside`, `shape-inside`, and `clip-path` should link to the same place, and it's better to mention `path()` together, to avoid confusing people. Or make all four links of `<basic-shape>` in the value definitions of these properties link to the same place, but add a note for `offset-path` which doesn't use the syntax of `path()` defined in [css-shapes-2].

Thanks.

Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/4270 using your GitHub account

Received on Tuesday, 3 September 2019 16:56:30 UTC