Dashboard tile drag: pointerdown-time engine snapshot can disagree with live engine reads mid-gesture
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
- Tipo di issue
- Bug
- Chiarezza
- Da chiarire
- Stato di attività
- Tranquilla
- Stack tecnologico
- typescript
- Ambito
- frontend
Direzione di ricerca
Leggi src/ui/dashboard-tile-gestures.ts, in particolare wireTileDrag e il relativo commento del modulo, quindi esegui i casi di caratterizzazione in tests/unit/dashboard.test.ts e tests/unit/dashboard-tile-gestures.test.ts. L’issue documenta un’incoerenza esistente del motore durante un gesto, ma lascia fuori ambito il comportamento desiderato; per completarla è necessaria una decisione dei maintainer, seguita da un’implementazione concordata e da test aggiornati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
In src/ui/dashboard-tile-gestures.ts's wireTileDrag, a drag gesture snapshots the active engine once, at pointerdown, into liveReflow (line 393: const liveReflow = deps.activeEngine() === 'grafana-grid';), which then governs the reflow/hit-test path for the rest of that one gesture. By contrast, deps.renderedSurface(tileId) — used for drop-target styling (setDrop, lines 449 and 451) and home-rect capture (beginMove, line 505) — is read live on every call; dashboard.ts's own implementation of that dependency re-checks its current engine each time, not just at gesture start. An engine flip mid-drag (e.g. an unrelated change-layout command dispatched while a move is active — nothing cancels an in-flight gesture on a command) makes the two disagree: the gesture keeps following whichever reflow path was active at pointerdown, while surface lookups made during the same gesture see the new engine.
Pinned by: the app-level characterization test "an engine flip mid-drag leaves the gesture on its GESTURE-START reflow path, not the live engine's" in tests/unit/dashboard.test.ts's "tile gesture concurrency characterization" block, and directly at the controller-unit level by "activeEngine() is read fresh at POINTERDOWN (not memoized at controller construction), and stays frozen for the rest of that one gesture while renderedSurface() keeps reading live" in tests/unit/dashboard-tile-gestures.test.ts.
Pre-existing latent inconsistency predating #589 (#593 refactor umbrella); wave 2's extraction preserved it verbatim (documented in the module's own doc comment, lines 17-26) rather than "fixing" it during a pure structural move. Out of scope here — filing for a deliberate follow-up decision.
- Lingua principale
- TypeScript
- Stelle
- 8
- Fork
- 2
- Merge medio
- 1h 34m
- PR unite (30g)
- 6
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Altinity/altinity-sql-browser
-
inbox
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
Altinity/altinity-sql-browser#605 ·
-
inbox
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
Altinity/altinity-sql-browser#509 ·
-
inbox
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
Altinity/altinity-sql-browser#489 ·
-
flamegraph Apertaenhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
Altinity/altinity-sql-browser#684 ·
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 68/100
Altinity/altinity-sql-browser#680 · 2 commenti ·
Tutte le issue di Altinity/altinity-sql-browser
Issue simili
-
Type/Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
OpenNSW/nsw-srilanka#497 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
0xMiden/bridge-portal#132 ·
-
react-doctor severity:warning tech-debt
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
digidem/comapeo-cloud-app#403 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100