Skip to content

repo-header-info - Restore feature#8869

Merged
fregante merged 3 commits into
refined-github:mainfrom
kovsu:fix-2
Jan 22, 2026
Merged

repo-header-info - Restore feature#8869
fregante merged 3 commits into
refined-github:mainfrom
kovsu:fix-2

Conversation

@kovsu
Copy link
Copy Markdown
Member

@kovsu kovsu commented Jan 14, 2026

@kovsu kovsu added the bug label Jan 14, 2026
Comment on lines 44 to 55
if (isPrivate && !elementExists('.octicon-lock', repoLink)) {
repoLink.append(
<LockIcon className="ml-1" width={12} height={12} />,
);
}

// GitHub may already show this icon natively, so we match its position
if (isFork && !elementExists('.octicon-repo-forked', repoLink)) {
repoLink.append(
<RepoForkedIcon className="ml-1" width={12} height={12} />,
);
}
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I don't know if we can drop these code. It seems that GitHub had already supported it.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We kept it because GitHub was inconsistent at the time. You can open a new PR to drop these for now and we can restore the feature if GitHub still fails

Comment thread source/features/repo-header-info.tsx Outdated
Comment thread source/features/repo-header-info.tsx Outdated
@fregante fregante merged commit f54d72b into refined-github:main Jan 22, 2026
8 checks passed
@kovsu kovsu deleted the fix-2 branch January 22, 2026 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants