coreui / coreui-pro / 33621617807

94%

main: 94%

push

github

fix(SectionInput, NumberInput, OTPInput): remove every listener in dispose() (#1049)

SectionInput kept its form `submit` listener past dispose() and stacked
another one on every re-initialization; a disposed instance then threw in
the microtask on forms with `data-coreui-validate`. The handler is stored
per instance, registered on the remembered form and removed in dispose()
by callable, so a sibling field in the same form keeps its own.

NumberInput registered the `input` listener and the document pointer
listeners without the component namespace and never removed them, so
typing after dispose() threw and every instance left four document
listeners behind. All of them carry the namespace now, the document ones
are removed in dispose() by callable, the button listeners are removed
before the buttons are, and the document `blur` listener is gone — blur
does not bubble, so it never fired.

OTPInput's paste listener lacked the namespace, so it survived dispose()
and threw on the nulled config after re-initialization. `update({ linear:
false })` also left `tabindex="-1"` on empty slots; every slot returns to
the tab order now.

Workspace audit v6-js-pre-alpha-audit-2026-08 §2.

4769 of 5356 branches covered (89.04%)

Branch coverage included in aggregate %.

33 of 33 new or added lines in 3 files covered. (100.0%)

9083 of 9502 relevant lines covered (95.59%)

392.98 hits per line