fix(ui): Escape that closes a menu also cancels an in-flight Move
I maintainer di solito rispondono entro 1 giorno
Una pull request collegata è già stata integrata.
- #223 di @cristim — integrata
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- Mezza giornata
- Idoneità per principianti
- 62/100
Direzione di ricerca
The Escape cancel is gated only on !files.is_modal() in src/app/panels.rs (around lines 31-33), so start with cancel_on_escape there and find how shortcuts check for an open popup (Popup::is_any_open). Add the same popup check to the shared native path. Done means Escape with the File menu open closes only the menu while Move stays active and the status is unchanged, a second Escape still cancels, and the Status Line T1 pinned test is updated. The regression test should use real key events.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem
Pressing Escape while any menu or popup is open (for example the File menu) also runs the editor's Escape handling (cancel_on_escape): it cancels an in-flight Move and rewrites the status to "Edit cancelled". The user only meant to close the menu.
Current behavior
src/app/panels.rs (around lines 31-33) gates the Escape cancel only on !files.is_modal(), not on whether an egui popup/menu is open (Popup::is_any_open). Same on main. Found while testing the Status Line menu-set combo; the Status Line T1 test pins the current behavior so a fix flips it deliberately.
Expected
Escape with a menu or popup open closes the popup only; Move and drafts are untouched and the status is unchanged. Escape with no popup open keeps the current cancel behavior.
Acceptance
- Gate the Escape cancel on no open popup/menu (same predicate shortcuts use), in the shared native path.
- Regression test with real key events: Move in flight, open File menu, Escape -> menu closes, Move still active, status unchanged; second Escape cancels as today. Update the Status Line T1 pinned test intentionally.
- Guard ledger note if Escape handling is documented there.
- Lingua principale
- Rust
- Stelle
- 4
- Fork
- 2
- Merge medio
- 9h 11m
- PR unite (30g)
- 181
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi 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 cristim/AztecCraft
-
A refused multi-object Delete does not name the refusing nodeForse già presa @RafaelMendes1407 l’ha presa 1 giorno fa. Apertaarea/mcp area/modeling effort/s impact/few priority/p3 severity/low triaged type/bug urgency/this-quarter
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
cristim/AztecCraft#336 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
InvalidInstancePath says 'face target' for transform instance pathsForse già presa @RafaelMendes1407 l’ha presa 1 giorno fa. Apertaarea/mcp area/selection effort/s impact/few priority/p3 severity/low triaged type/bug urgency/this-quarter
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
cristim/AztecCraft#327 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Derive the behind-camera ring test pixel by scan instead of a fixed pointForse già presa @RafaelMendes1407 l’ha presa 1 giorno fa. Apertaarea/viewport effort/s impact/few priority/p3 severity/low triaged type/chore urgency/this-quarter
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
cristim/AztecCraft#322 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Uniform Scale hover and press labels differ (World Uniform Scale vs Uniform Scale)Forse già presa @RafaelMendes1407 l’ha presa 1 giorno fa. Apertaarea/ui-parity effort/s impact/few priority/p3 severity/low triaged type/bug urgency/this-quarter
Difficoltà 2/5 1-3 ore Idoneità per principianti 69/100
cristim/AztecCraft#318 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
test: pin missing_move_viewport's cancel branch for an active moveForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertaarea/viewport effort/s impact/internal priority/p3 severity/low triaged type/chore urgency/this-quarter
Difficoltà 3/5 1-2 giorni Idoneità per principianti 67/100
cristim/AztecCraft#346 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di cristim/AztecCraft
Issue simili
-
comp-view potential bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 80/100
ClickHouse/ClickHouse#128410 ·
I maintainer di solito rispondono entro 1 giorno
-
Save As onto the project's own .snxprj overwrites it with a schematic and the project can no longer openForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertaarea: app data-loss priority: high type: bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno
-
lan_bot: 13 of 15 tests fail on main (minecraft:dialog entries carry data with the known pack)Forse già presa Una pull request collegata a questa issue è aperta o già unita. Apertaskip-release
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
horizon-rust/horizon#5505 ·
I maintainer di solito rispondono entro 1 giorno
-
area: cloud S3: minor
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
manaflow-ai/cmux#29709 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
area:mem good first issue priority:low type:docs
Difficoltà 2/5 1-3 ore Idoneità per principianti 90/100
I maintainer di solito rispondono entro 1 giorno