Skip to content

fix: don't dim draft PRs so they're easy to spot #28

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
facundofarias merged 1 commit into from
Jul 1, 2026
Merged

fix: don't dim draft PRs so they're easy to spot #28

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion src/popup/components/PRItem.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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;

@chatgpt-codex-connector chatgpt-codex-connector Bot Jul 1, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Keep stale drafts undimmed

When a draft is older than stalePRDays, isStale is still true, so the row continues to receive opacity-50 even though this change says drafts should not be dimmed. This leaves long-running own draft PRs grayed out in the exact case the fix is trying to avoid; apply the stale dimming only when !pr.isDraft if drafts should always stay visually prominent.

Useful? React with 👍 / 👎.

const isMergeable = !pr.isDraft && !pr.isMerged && !pr.hasConflicts && pr.ciStatus !== 'failed';

async function handleMerge() {
Expand Down
Loading