[css-properties-values-api] Example 2: should explain more clearly why `inherit` instead of `green` in description

[Marat-Tanalin]

The example 2 of CSS Properties and Values API Level 1 draft contains the following code:

.thing {
    --my-color: green;
    --my-color: url("not-a-color");
    color: var(--my-color);
}

But then description says that the color property is set to inherit instead of green. So either this is a typo, and inherit in the description should be replaced with the correct green used in the code, or the description should be more clear on why the property is set to inherit instead of green.

[FremyCompany]

This is accurate. --my-color has no type, so its value becomes url("...").
As a result, color becomes url("...") which is invalid at computation time, so color reverts to its initial value, which is inherit.

[FremyCompany]

I just had a read, and this is exactly what the spec says. Not sure what you think was not clear.

Closing the issue but pinging @tabatkins just for the sake of it, maybe he can improve the wording a bit. Why not something like:

At this stage of the css pipeline (computation time), the only available fallback is the initial value of the property, which in the case of "color" is "inherit".

[Marat-Tanalin]

Thanks, François. After your explanation, the example and its description are more clear. The description in the draft could probably be updated with a more clear text like in your first comment.

[shans]

I've reopened with an updated description so we can improve the prose here.

Hmm, this should probably be filed in the Houdini repository.

[shans]

w3c/css-houdini-drafts#209