Skip to content

Use distinct git icons for PR state in sidebar threads#229

Merged
BunsDev merged 1 commit into
mainfrom
okcode/git-cycle-thread-labels
Apr 4, 2026
Merged

Use distinct git icons for PR state in sidebar threads#229
BunsDev merged 1 commit into
mainfrom
okcode/git-cycle-thread-labels

Conversation

@BunsDev

@BunsDev BunsDev commented Apr 4, 2026

Copy link
Copy Markdown
Member

Summary

  • Thread PR indicators in the sidebar now use state-specific icons instead of a single GitPullRequestIcon for all states
  • OpenGitPullRequestIcon (green), ClosedXCircleIcon (gray), MergedGitMergeIcon (violet)
  • Matches the icon mapping already used in the PR Review panel for visual consistency

Test plan

  • Open the sidebar with threads that have associated PRs in different states (open, closed, merged)
  • Verify each thread shows the correct icon and color for its PR state
  • Confirm clicking the icon still opens the PR link

🤖 Generated with Claude Code

Thread PR indicators now show GitPullRequestIcon for open, XCircleIcon
for closed, and GitMergeIcon for merged — matching the PR Review panel.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel

vercel Bot commented Apr 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview Apr 4, 2026 6:43pm
v0-compute-the-platform-to-build Ready Ready Preview, Open in v0 Apr 4, 2026 6:43pm

@github-actions github-actions Bot added the vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. label Apr 4, 2026
@BunsDev BunsDev merged commit 50aee00 into main Apr 4, 2026
9 of 10 checks passed
@BunsDev BunsDev deleted the okcode/git-cycle-thread-labels branch April 6, 2026 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant