Remove <ESC> for quit in dashboard #98
Description
<ESC> universally means "back", not "exit the program." When already at the root view, there's nowhere to go back to, so ESC doing nothing is the correct behavior.
In the TUI <ESC> is used in 6 of 8 contexts to mean "go back / cancel / dismiss":
| Context | <ESC> Action |
Meaning |
|---|---|---|
| DashboardFilter | ClearFilter |
Cancel/dismiss overlay |
| WorktreeFilter | ClearFilter |
Cancel/dismiss overlay |
| DashboardInput | ExitInputMode |
Exit modal sub-mode |
| DiffNormal | CloseDiff |
Go back to parent view |
| Patch | ExitPatchMode |
Go back to parent view |
| Comment | CancelComment |
Cancel/dismiss overlay |
| Dashboard | Quit |
Exit application |
| Worktree | Quit |
Exit application |
The transition from "go back one level" to "kill the app" is invisible and instant.
q and Ctrl+C already cover quit. q is the standard TUI quit key (vim, less, top, htop,
lazygit, etc). Ctrl+C is the universal stop signal. ESC adds no value beyond an accidental exit path.