Re: [csswg-drafts] [css-box-4][css-backgrounds-4][css-shapes-1] Replace <box> definition with <visual-box> (#9505)

Ok, I reverted the namespace change, and introduced an informal `<box>` definition (without a production rule).

I also simplified the production rules to also fix the namespaces defined for each individual keyword.

CSS Shapes 1 defines `<shape-box>` with the same keywords than `<layout-box>`. I leave it up to the spec authors to replace it with `<layout-box>`, as well as in CSS Masking 1, which includes `<shape-box>` in `<geometry-box>`.

As a recap with flattened production rules:

```
  <visual-box> = content-box | padding-box | border-box
  <layout-box> = content-box | padding-box | border-box | margin-box
   <shape-box> = content-box | padding-box | border-box | margin-box
<geometry-box> = content-box | padding-box | border-box | margin-box | fill-box | stroke-box | view-box
   <paint-box> = content-box | padding-box | border-box | fill-box | stroke-box
   <coord-box> = content-box | padding-box | border-box | fill-box | stroke-box | view-box
```

Proposed:

```
  <visual-box> = content-box | padding-box | border-box
  <layout-box> = <visual-box> | margin-box
   <shape-box> = <visual-box> | margin-box
<geometry-box> = <shape-box> | fill-box | stroke-box | view-box
  <paint-box>  = <visual-box> | fill-box | stroke-box
  <coord-box>  = <paint-box> | view-box
```

Ideally (imo):

```
  <visual-box> = content-box | padding-box | border-box
  <layout-box> = <visual-box> | margin-box
   <paint-box> = <visual-box> | fill-box | stroke-box
   <coord-box> = <paint-box> | view-box
<geometry-box> = <layout-box> | fill-box | stroke-box | view-box
```

-- 
GitHub Notification of comment by cdoublev
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/pull/9505#issuecomment-1772675452 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Friday, 20 October 2023 12:44:41 UTC