Skip to content

Prevent chart pointer update loops - #2456

Open
ejsmith wants to merge 2 commits into
mainfrom
issue/fix-chart-pointer-depth
Open

Prevent chart pointer update loops#2456
ejsmith wants to merge 2 commits into
mainfrom
issue/fix-chart-pointer-depth

Conversation

@ejsmith

@ejsmith ejsmith commented Aug 10, 2026

Copy link
Copy Markdown
Member

Summary

  • make chart tooltips ignore pointer events so a cursor-following tooltip cannot steal and immediately return chart hover during drag
  • let LayerChart perform its built-in brush reset instead of resetting the same brush a second time from application callbacks
  • close an open stack detail sheet before saved-view pathname navigation rehydrates the stack list
  • cover both production reproductions: dragging the stack timeline and rapidly alternating saved views with stack details open

Root cause

The shared chart tooltip followed the cursor while remaining pointer-interactive. During a pressed drag, it could repeatedly enter the pointer path, hide or unmount, return hover to the chart, and recreate itself inside one reactive flush. Range-enabled dashboard charts also reset LayerChart's brush manually even though the wrapper already resets it after the callback.

Saved-view navigation was a separate overload path: the stack detail subtree remained mounted while each pathname change rehydrated filters, columns, and queries for a different saved view. Rapid navigation therefore updated the outgoing detail graph and incoming list graph together.

Verification

  • npm run validate
  • npm run build
  • focused Playwright stack-effects-chaos.e2e.ts, including a pressed multi-step timeline drag and 20 alternating saved-view navigations with a stack detail open
  • GitHub Actions

@ejsmith
ejsmith marked this pull request as ready for review August 10, 2026 21:04
@github-actions

Copy link
Copy Markdown

Code Coverage

Package Line Rate Branch Rate Complexity Health
Exceptionless.Core 75% 67% 9992
Exceptionless.AppHost 38% 40% 139
Exceptionless.Insulation 37% 35% 286
Exceptionless.Web 85% 69% 7024
Summary 78% (23466 / 30272) 67% (11013 / 16516) 17441

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