coreui / coreui-pro / 32466866916

94%

main: 94%

push

github

fix(Table): declare every value it draws with as a CSS variable (#812)

Cell padding, vertical alignment, border width and the group separator
colour were compiled into the rules, so nothing could reach them at
runtime. Stacked tables already read `--cui-table-cell-padding-y`,
`--cui-table-cell-padding-x` and `--cui-table-border-width`, which no rule
declared: every one of those reads was invalid at computed-value time, so
stacked rows rendered with zero padding and no line between them.

Emit the whole set through `defaults()` + `tokens()` on `.table`, and give
the striped, active and hover backgrounds a `--cui-table-*-bg-factor`
variable so their strength is adjustable without redefining the colour.
They now mix from `--cui-table-color` rather than `--cui-emphasis-color`,
which lets `table-variant()` drop six declarations: setting the colour,
background and border colour is enough for the states to follow.

`.table-sm` sets the padding variables on the table instead of overriding
the padding of every cell, so the modifier reaches stacked rows too.

Borders move to logical properties, so a vertical writing mode keeps the
separators on the block axis instead of turning them sideways.

4537 of 5095 branches covered (89.05%)

Branch coverage included in aggregate %.

8661 of 9055 relevant lines covered (95.65%)

399.88 hits per line