Skip to content

fix(tui): clarify working copy actions - #40935

Open
kitlangton wants to merge 1 commit into
v2from
confirm-action-labels
Open

fix(tui): clarify working copy actions#40935
kitlangton wants to merge 1 commit into
v2from
confirm-action-labels

Conversation

@kitlangton

Copy link
Copy Markdown
Contributor

What

Replace ambiguous no / yes working-copy confirmation labels with actions that describe the result.

Before / After

Before: Move and forced-delete confirmations displayed no / yes, requiring users to infer what each choice would do.

After: Move uses cancel / move; forced deletion uses cancel / delete.

How

  • Add optional choice labels to DialogWorkspaceFileChanges.
  • Keep move labels as the defaults.
  • Supply deletion-specific labels from DialogMoveSession.

Scope

No confirmation behavior or default selection changes.

Testing

  • bun typecheck from packages/tui
  • bun run test test/cli/tui/dialog-select.test.tsx
  • Repository pre-push typecheck
  • git diff --check

@kitlangton

Copy link
Copy Markdown
Contributor Author

Demo

Before: The move confirmation uses no / yes.

labels-before.mp4

After: The same keyboard flow uses cancel / move.

labels-after.mp4

@kitlangton
kitlangton enabled auto-merge (squash) August 6, 2026 23:25
@kitlangton
kitlangton force-pushed the confirm-action-labels branch from dd703d0 to 85f03b2 Compare August 6, 2026 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant