Skip to content

fix: replace delete-on-hover with archive animation for merged PRs#112

Merged
amandal0903 merged 1 commit intomainfrom
remove-delete-with-archive
Apr 13, 2026
Merged

fix: replace delete-on-hover with archive animation for merged PRs#112
amandal0903 merged 1 commit intomainfrom
remove-delete-with-archive

Conversation

@amandal0903
Copy link
Copy Markdown
Collaborator

Summary

  • Hide the delete (trash) icon on hover for worktrees whose PR is already merged — the Archive button is the
    correct action for these
  • Add a smooth fade-out + height-collapse animation when archiving instead of the item vanishing abruptly

Test plan

  • Hover a worktree with a merged PR — no delete icon should appear
  • Hover a worktree without a merged PR — delete icon still appears as before
  • Click Archive on a merged-PR worktree — item should fade out and collapse over ~350ms, then be removed
  • Verify the main worktree row still has no delete or archive buttons

…animation

Merged-PR worktrees now only show the Archive action (no redundant
delete icon on hover). Archiving animates with a fade-out and height
collapse instead of disappearing abruptly.
@amandal0903 amandal0903 merged commit ca35281 into main Apr 13, 2026
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