Skip to content

Keep engaged plot drags continuous through the press position - #32

Merged
imakris merged 1 commit into
masterfrom
fix/plot-drag-origin-20261111
Nov 11, 2026
Merged

imakris merged 1 commit into
masterfrom
fix/plot-drag-origin-20261111

Conversation

@imakris

@imakris imakris commented Nov 11, 2026 •

Copy link
Copy Markdown
Member

Returning an engaged main-plot drag to its original press position discarded the returning movement and retained the preceding pan offset. Apply the click tolerance only while the gesture remains a click candidate, preserving the existing pan/clamp and click semantics.

Add sparse return and dense crossing event regressions with local and shared time axes, plus a check that genuine click slop still leaves the range unchanged. The pre-fix offscreen fixture reproduced press x=100 → move x=120 → return x=100 leaving [900,1900] instead of [1000,2000] (39 passing cases, one failing regression).

Independent source review found no material issues. All eight public CI jobs passed, including PlotInteractionItem in each configuration:

  • Windows: 26 tests without text rendering; 47 with text rendering.
  • Linux, macOS, and FreeBSD: 26 tests without text rendering and 46 with text rendering on each platform.

No application UI or visual/interaction-feel certification is claimed.

The initial click tolerance also discarded movement after a drag had engaged.
Returning the pointer to its press position therefore retained the preceding
pan offset. Apply tolerance only while the press remains a click candidate;
subsequent deltas continue through the existing pan and range-clamping paths.

Add event regressions for sparse return and dense crossing sequences with
local and shared time axes, no drag-generated click, and retained click slop.
The sparse sequence reproduced a [900,1900] range instead of [1000,2000]
before the fix. Final validation uses the configured public pull-request CI.
@imakris
imakris merged commit da5b4a0 into master Nov 11, 2026
12 checks passed
@imakris
imakris deleted the fix/plot-drag-origin-20261111 branch November 11, 2026 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant