Skip to content

feat: add text labels to resolve/unresolve buttons#68

Merged
tomasz-tomczyk merged 1 commit intomainfrom
feat/resolve-button-text-label
Mar 30, 2026
Merged

feat: add text labels to resolve/unresolve buttons#68
tomasz-tomczyk merged 1 commit intomainfrom
feat/resolve-button-text-label

Conversation

@tomasz-tomczyk
Copy link
Copy Markdown
Owner

Summary

Port from tomasz-tomczyk/crit#220.

  • Replace icon-only resolve/unresolve buttons with labeled pill buttons ("Resolve" / "Unresolve") for better discoverability and click target size
  • Unresolve button gets a green border to signal resolved state, replacing the separate "Resolved" badge
  • Resolve button stays subtle (no border) until the comment is actually resolved

Test plan

  • Visual check: resolve button on unresolved comments shows subtle "Resolve" label
  • Visual check: unresolve button on resolved comments shows green-bordered "Unresolve" pill
  • Verify light and dark themes both look correct

🤖 Generated with Claude Code

Port from tomasz-tomczyk/crit#220. Replace icon-only resolve/unresolve
buttons with labeled pill buttons showing "Resolve" or "Unresolve" text
next to the icon for better discoverability. The Unresolve button gets a
green border to signal resolved state, replacing the separate "Resolved"
badge.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@tomasz-tomczyk tomasz-tomczyk merged commit 4e58e69 into main Mar 30, 2026
1 check passed
@tomasz-tomczyk tomasz-tomczyk deleted the feat/resolve-button-text-label branch March 30, 2026 10:07
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