naver / billboard.js / 24556718728

94%

push

github

fix(zoom): fix drag zoom boundary release and circle transition desync 

Two related fixes for drag-zoom UX (issue #4131 and follow-up):

1. clampPointer in initZoomBehaviour (zoom.ts)
   - Mouse release outside the chart boundary silently dropped the zoom
     because the resulting end value fell outside scale domain, failing
     withinRange() in chart.zoom().
   - Clamp start/drag/end pointer values to [0, state.width/height] when
     axis.x.extent is not configured, so onzoomend fires and the zoom is
     applied even when the cursor leaves the plot area.

2. Distinct transition name for circle cx/cy (point.ts)
   - redrawCircle scheduled cx/cy and opacity transitions with the same
     name "t" on overlapping selections. D3-transition interrupts prior
     same-named transitions on the same element, so the cx/cy animation
     was cancelled before commit — circles stayed at pre-zoom positions
     while line and axis moved correctly (visual desync).
   - Use `${t}-pos` for cx/cy so it runs independently of the opacity
     transition, restoring synchronized animation.

Add regression tests for both in test/interactions/zoom-spec.ts.

Close #4131

6907 of 7680 branches covered (89.93%)

Branch coverage included in aggregate %.

6 of 6 new or added lines in 1 file covered. (100.0%)

8709 of 9000 relevant lines covered (96.77%)

20754.95 hits per line