coreui / coreui-pro / 32470691739

94%

main: 94%

push

github

fix(Grid): declare the grid variables so a nested grid keeps its own layout (#820)

`.grid` read `--cui-rows`, `--cui-columns` and `--cui-gap` through a `var()`
fallback and declared none of them. A custom property that is never declared
inherits, so a `.grid` nested inside a three-column grid silently became a
three-column grid — measured in Chromium: 3 columns nested inside 3, where
declaring the variables gives the documented 12. Our own docs worked around
it by setting `--cui-columns: 12` on the nested grid in the nesting example.

Declaring all three on `.grid` makes each grid self-contained: a component
built on `.grid` keeps its layout wherever it is dropped, and the variables
become visible in dev tools instead of existing only as the fallback arm of
a `var()` call.

The nesting page was written around the old behaviour and described the
inheritance as the feature, so it is rewritten against what the CSS now
does, and the customizing table stops calling the values fallbacks.

Correct one claim while here: the `cq-*` section said `$enable-container-queries`
was removed. It is not — it is accepted and ignored, and has to stay until v7
because the shared docs engine passes it in its `with()` block for every
edition, where an unknown variable is a hard Sass error.

4537 of 5095 branches covered (89.05%)

Branch coverage included in aggregate %.

8661 of 9055 relevant lines covered (95.65%)

399.9 hits per line