[css-box-4][css-backgrounds-4][css-shapes-1] Cleanup <*-box> definitions

[cdoublev]

Follow-up on 7dc439c, fixing speced/bikeshed#2679.

The commit removed the production (which became undefined) and replaced references with , but some references remained in other specs (including CSS Box, which defines ).

[cdoublev]

Hmm, /fill-box and other links are incorrect. I am not sure how to fix them, except by re-introducing = | | ....

[cdoublev]

Ok, I reverted the namespace change, and introduced an informal 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 with the same keywords than . I leave it up to the spec authors to replace it with , as well as in CSS Masking 1, which includes in .

As a recap with flattened production rules:

   = content-box | padding-box | border-box
   = content-box | padding-box | border-box | margin-box
    = content-box | padding-box | border-box | margin-box
 = content-box | padding-box | border-box | margin-box | fill-box | stroke-box | view-box
    = content-box | padding-box | border-box | fill-box | stroke-box
    = content-box | padding-box | border-box | fill-box | stroke-box | view-box

Proposed:

   = content-box | padding-box | border-box
   =  | margin-box
    =  | margin-box
 =  | fill-box | stroke-box | view-box
    =  | fill-box | stroke-box
    =  | view-box

Ideally (imo):

   = content-box | padding-box | border-box
   =  | margin-box
    =  | fill-box | stroke-box
    =  | view-box
 =  | fill-box | stroke-box | view-box

[w3cbot]

svgeesus marked as non substantive for IPR from ash-nazg.

[tabatkins]

Your for values are broken. Note that Bikeshed doesn't actually process for values in any way, they're just opaque strings; the fact that includes doesn't automatically make the values that are for="" also for , etc.

So you need to accumulate all the <*-box> variants that include each keyword in the for values for that keyword.

[cdoublev]

the fact that includes doesn't automatically make the values that are for="" also for , etc.

I think this is the way w3c/reffy will process it, assuming is defined for (and other productions), which I forgot to define.

Here is the (simplified) representation of the data I expect to find in w3c/webref:

  "values": [
    {
      "name": "box",
      "values": [
        {
          "name": "coord-box",
          "value": " | view-box",
          "values": [
            {
              "name": "paint-box",
              "value": " | fill-box | stroke-box",
              "values": [
                {
                  "name": "visual-box",
                  "value": "content-box | padding-box | border-box",
                  "values": [{ "name": "content-box" }, { "name": "padding-box" }, { "name": "border-box" }]
                },
                { "name": "fill-box" },
                { "name": "view-box" }
              ]
            }
          ]
        },
        {
          "name": "geometry-box",
          "value": " | fill-box | stroke-box | view-box",
          "values": [
            {
              "name": "layout-box",
              "value": " | margin-box",
              "values": [
                {
                  "name": "visual-box",
                  "value": "content-box | padding-box | border-box",
                  "values": [{ "name": "content-box" }, { "name": "padding-box" }, { "name": "border-box" }]
                },
                { "name": "margin-box" }
              ]
            },
            { "name": "fill-box" },
            { "name": "stroke-box" },
            { "name": "view-box" }
          ]
        },
        {
          "name": "layout-box",
          "value": " | margin-box",
          "values": [
            {
              "name": "visual-box",
              "value": "content-box | padding-box | border-box",
              "values": [{ "name": "content-box" }, { "name": "padding-box" }, { "name": "border-box" }],
            },
            { "name": "margin-box" }
          ]
        },
        {
          "name": "paint-box",
          "value": " | fill-box | stroke-box",
          "values": [
            {
              "name": "visual-box",
              "value": "content-box | padding-box | border-box",
              "values": [{ "name": "content-box" }, { "name": "padding-box" }, { "name": "border-box" }],
            },
            { "name": "fill-box" },
            { "name": "view-box" }
          ]
        },
        {
          "name": "visual-box",
          "value": "content-box | padding-box | border-box",
          "values": [{ "name": "content-box" }, { "name": "padding-box" }, { "name": "border-box" }],
        },
      ]
    },
  ]

Would it be ok? Cc @tidoust for a review.

Actually, I do not care about the context (namespace) of a production because I am only interested by its syntax (value), assuming I am right to consider it context-insensitive.

And I am not sure it is right to restrict eg. for all other <-*box> productions. One may think that cannot be used outside of these contexts.

I do not even need references to be removed from CSS Box. I just need the changes in CSS Backgrounds 4 and CSS Shapes. So feel free to make whatever changes you think are appropriate.

[cdoublev]

I went ahead and defined each keyword with the appropriate contexts. Bikeshed does not report any errors.