Skip to content

Refactor barogram-map synchronization - #960

Open
JEK58 wants to merge 2 commits into
mainfrom
codex/refactor-barogram-map-sync
Open

Refactor barogram-map synchronization#960
JEK58 wants to merge 2 commits into
mainfrom
codex/refactor-barogram-map-sync

Conversation

@JEK58

@JEK58 JEK58 commented Jul 29, 2026

Copy link
Copy Markdown
Member

Summary

  • replace the Chart.js tooltip callback event bus with a requestAnimationFrame-coalesced timestamp cursor
  • synchronize map markers by nearest timestamp across independently sampled main and Airbuddy tracks
  • update Leaflet layers imperatively and diff Airbuddy layers without recreating the main track or static markers
  • simplify replay, decimate chart rendering, cache METAR selection, and remove obsolete synchronization composables

Why

The previous tooltip-driven synchronization performed Chart.js hit-testing, tooltip callbacks, and deep Vue/Leaflet updates for each pointer event. This refactor keeps chart pointer handling outside Chart.js and only moves changed markers.

Validation

  • pnpm typecheck
  • focused ESLint
  • pnpm exec cypress run --spec cypress/e2e/helpers/flightCursorHelpers.cy.js
  • pnpm build
  • live frontend check against flight 86625 through a local live-API proxy

@JEK58
JEK58 marked this pull request as ready for review July 29, 2026 07:15
@JEK58
JEK58 requested a review from KaiWissel July 29, 2026 07:22
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