Cmd+← / back navigation in Directory Preview discards unsaved file edits without prompt
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 68/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Área
- frontend
Línea de trabajo
Start at the Directory Preview back-navigation handler used by Cmd+← and the toolbar back arrow, then trace how the current file editor exposes unsaved state. Reproduce the edit-and-navigate sequence and verify that dirty files show Save, Discard, and Cancel choices before leaving, while clean files still navigate immediately.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
When browsing files in Directory Preview mode and editing a file (e.g., a markdown file), pressing Cmd+← (or clicking the back arrow) navigates back to the directory listing and silently discards all unsaved edits — no confirmation, no prompt.
Steps to Reproduce
- Open a directory in Wave Terminal (Directory Preview)
- Click a
.mdfile to open it for inline editing - Type some content but do not press
Cmd+S - Press
Cmd+←or click the back arrow in the toolbar
Expected: A dialog appears: "You have unsaved changes. Save before leaving?" with options to Save / Discard / Cancel.
Actual: Navigation back happens immediately, all unsaved edits are lost.
Context
- There is no shortcut or gesture that currently closes the editor with a save prompt — the dirty-state check simply doesn't exist on the Directory Preview back-navigation path
- This is a data-loss risk for anyone using the built-in file editing in directory preview
Suggested Fix
Add a dirty-state check to the Directory Preview back-navigation handler. When there are unsaved edits in the current file preview/edit view, prompt the user before navigating back.
Related
- Existing "confirm close" settings (
tab:confirmclose,window:confirmclose,app:confirmquit) only cover tab/window/app-level close, not intra-widget navigation back ~/.waveterm/config/keybindings.jsonallows remapping keys but can't add conditional logic (e.g., "check dirty state before executing back command")
- Lenguaje dominante
- Go
- Estrellas
- 22.4k
- Forks
- 1.1k
- Merge medio
- 3 d 13 h
- PR fusionados (30 d)
- 13
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de wavetermdev/waveterm
-
pwsh: encodeEnvVarsForPowerShell rejects Windows-standard env var names like ProgramFiles(x86), breaking wsh token on every pwsh blockPosiblemente ocupada @vortsghost2025 la tomó hace 27 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
wavetermdev/waveterm#3481 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
wavetermdev/waveterm#3435 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
wavetermdev/waveterm#3432 ·
Los mantenedores suelen responder en 1 día
-
enhancement triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
wavetermdev/waveterm#3431 ·
Los mantenedores suelen responder en 1 día
-
[Bug]: web blocks become unresponsive when switching focus between two of themPosiblemente ocupada @jameswolensky la tomó hace 85 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
wavetermdev/waveterm#3428 ·
Los mantenedores suelen responder en 1 día
Todos los issues de wavetermdev/waveterm
Issues similares
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 90/100
FootprintAI/Containarium#2338 ·
Los mantenedores suelen responder en 1 día
-
[Bug]: core doesn't build standalone on dev since a17068054 (go-mp3 require dropped, go.sum pruned)Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
SagerNet/sing-openvpn#11 ·
-
priority: P3 type: devops
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
jiegui2025/hwspec#57 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
columnar-tech/dbc#513 ·
Los mantenedores suelen responder en 1 día