From 2dcc6dfd911e61347b9256c4aa766b166d36e006 Mon Sep 17 00:00:00 2001 From: Facundo Farias Date: Wed, 1 Jul 2026 12:02:07 +0200 Subject: [PATCH] fix: don't dim draft PRs so they're easy to spot MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Drafts were dimmed (opacity-50) like stale/reviewed/merged PRs, which made users think their own drafts weren't listed at all — they were just faint and sorted low. A draft is your own active work-in-progress, not a de-prioritized PR, and it already carries a ✍️ marker. Drop isDraft from the dim condition so drafts render at full opacity. A merged former-draft still dims via isMerged. Fixes #25 Co-Authored-By: Claude Opus 4.8 (1M context) --- src/popup/components/PRItem.tsx | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/popup/components/PRItem.tsx b/src/popup/components/PRItem.tsx index 0dc8753..ab4923c 100644 --- a/src/popup/components/PRItem.tsx +++ b/src/popup/components/PRItem.tsx @@ -121,7 +121,10 @@ export default function PRItem({ pr, stalePRDays, pinned, onMerged, focused, sta }, [pr.id, pr.headSha, pr.unresolvedCommentCount]); const timeAgo = getTimeAgo(pr.updatedAt); const isStale = stalePRDays > 0 && (Date.now() - new Date(pr.updatedAt).getTime()) > stalePRDays * 86400000; - const isDimmed = (pr.hasReviewed && !pr.isAuthor) || isStale || pr.isBot || pr.isMerged || pr.isDraft; + // Drafts are NOT dimmed: a draft is your own active work-in-progress, not a + // de-prioritized PR like stale/reviewed/merged ones. The ✍️ marker already + // flags it, and dimming made users think their drafts weren't listed (#25). + const isDimmed = (pr.hasReviewed && !pr.isAuthor) || isStale || pr.isBot || pr.isMerged; const isMergeable = !pr.isDraft && !pr.isMerged && !pr.hasConflicts && pr.ciStatus !== 'failed'; async function handleMerge() {