coreui / coreui-pro / 33621622592

93%

main: 94%

Build Type

push

github

Committed by web-flow

Commit Message

fix(Stepper, RangeSlider, LoadingButton): keyboard capture, reset after finish, step math, shared dispose and stale timers (#1047)

Stepper:
- the keydown listener is delegated to `.stepper-step-button`, so Arrow/Home/End
  typed in a form control inside a step pane keep their native behaviour instead of
  being prevented and moving focus to an edge step button
- `reset()` works on every step button, so it no longer turns into a no-op after
  `finish()` disabled them all; the linear disabled state is recomputed at the end

RangeSlider:
- `_roundToStep()` anchors the step grid at `min`, clamps to `[min, max]`, steps
  down when the nearest grid point overshoots `max` (as the native input does),
  fixes the result to the precision of the step and `min`, and only clamps for a
  non-numeric step such as `"any"`; a pointer beyond the track goes through the
  same rounding
- the document/window handlers are per-instance functions, so `dispose()` removes
  only its own listeners instead of every slider's under the shared namespace, and
  `update()` no longer stacks duplicates

LoadingButton:
- the `timeout` timer id and the start-delay timer id are stored and cleared in
  `stop()` and in `dispose()`, so a manual `stop()` before the timeout no longer
  gets a stale `stop()` (and a `remove()` on a null spinner), and a disposed
  instance is never touched by its timers
- `stop()` is a no-op when the button is not loading
- the redundant `Data.set()` in the constructor is gone: the base constructor
  already registers the instance, and with a selector string this created a
  string-keyed entry `dispose()` could never remove

Found by workspace audit v6-js-pre-alpha-audit-2026-08 §1.

Coverage Stats

4763 of 5352 branches covered (88.99%)

Branch coverage included in aggregate %.

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

1 existing line in 1 file now uncovered.

9067 of 9486 relevant lines covered (95.58%)

391.67 hits per line