naver / billboard.js / 24556718728 / 3

94%

master: 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 %.

8709 of 9000 relevant lines covered (96.77%)

6918.2 hits per line