Problem
When using the TUI interface, I noticed that the Message Actions dialog defaults to the "Revert" option. This is a potential UX issue because Revert is a destructive operation that undoes messages and file changes. If I accidentally hit Enter without thinking, it could cause unintended data loss or changes that I didn't mean to apply.
Why This Matters
Safe operations like "Copy" should be the first option since they're non-destructive. Most users expect to be able to safely interact with a dialog without accidentally triggering dangerous actions.
What Should Happen
The message actions should be reordered to put the safe "Copy" operation first, so it becomes the default selection. This way:
- Users can safely press Enter on the first option
- Destructive operations like "Revert" come at the end as expected
Affected Code
The options are defined in packages/tui/src/routes/session/dialog-message.tsx
Problem
When using the TUI interface, I noticed that the Message Actions dialog defaults to the "Revert" option. This is a potential UX issue because Revert is a destructive operation that undoes messages and file changes. If I accidentally hit Enter without thinking, it could cause unintended data loss or changes that I didn't mean to apply.
Why This Matters
Safe operations like "Copy" should be the first option since they're non-destructive. Most users expect to be able to safely interact with a dialog without accidentally triggering dangerous actions.
What Should Happen
The message actions should be reordered to put the safe "Copy" operation first, so it becomes the default selection. This way:
Affected Code
The options are defined in
packages/tui/src/routes/session/dialog-message.tsx