Skip to content

Conversation

@mafredri
Copy link
Member

@mafredri mafredri commented Nov 28, 2025

Reduce from 3 confirmation stages to 2 by removing the redundant
"resources" stage. The final button now shows "Delete N tasks and M
workspaces" directly, so users still see what will be deleted.

Also add a Cancel button to match the single task delete dialog UX.

Refs #20905

Slack thread for context: https://codercom.slack.com/archives/C0992H8HGCS/p1763990324171349


🤖 This change was written by Claude Opus 4.5 Thinking using mux and reviewed by a human 🏂

This change addresses feedback on the tasks bulk delete feature:

1. Removed the third confirmation stage (resources stage) - the flow
   now goes directly from consequences to tasks review, then delete
2. Added Cancel button to the dialog by removing hideCancel prop -
   this matches the single task delete confirmation dialog UX
3. Moved workspace count display to the final confirm button text
   so users still see what will be deleted

The simplified flow reduces friction while still providing adequate
warning about the destructive nature of the action.
This change addresses feedback on the tasks bulk delete feature:

1. Removed the third confirmation stage (resources stage) - the flow
   now goes directly from consequences to tasks review, then delete
2. Added Cancel button to the dialog by removing hideCancel prop -
   this matches the single task delete confirmation dialog UX
3. Moved workspace count display to the final confirm button text
   so users still see what will be deleted

The simplified flow reduces friction while still providing adequate
warning about the destructive nature of the action.
@mafredri mafredri marked this pull request as ready for review November 28, 2025 16:11
Copy link
Contributor

@ssncferreira ssncferreira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🏂
Could you maybe include a link to the internal slack thread, for future reference?

display_name: "Update Documentation",
initial_prompt: "Update documentation for the new features",
// Intentionally null to test that only 2 workspaces are shown in review resources stage
// Intentionally null to test that only 2 workspaces are shown
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@mafredri mafredri requested review from jaaydenh and removed request for aslilac December 3, 2025 14:56
@mafredri mafredri merged commit 2f82928 into main Dec 4, 2025
62 of 64 checks passed
@mafredri mafredri deleted the tasks-bulk-delete-ui-improvements branch December 4, 2025 08:46
@github-actions github-actions bot locked and limited conversation to collaborators Dec 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants