coreui / coreui-pro / 32481278796

93%

main: 94%

push

github

feat(Mixins): let transition-props() emit transition-behavior (#829)

`display` and `content-visibility` are discrete properties: they only take
part in a transition with `transition-behavior: allow-discrete`, which is what
keeps an element laid out until its fade-out ends. Five components need it —
menu, popup, toast, accordion and collapse — and none could use the longhand
mixin, because it had no way to emit that declaration. They write the
`transition` shorthand instead.

The cost is not the shorthand itself, it is what the longhand gives up: it
keeps the property list in a custom property, so a consumer adds a property to
the animation by extending one token. With the shorthand they have to restate
the whole declaration.

The file is now byte-identical to upstream's, `$behavior` parameter and
`transition-longhands()` helper included; nothing in it was ours to keep.
No caller passes the argument yet, so the compiled CSS is unchanged.

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