8000 `repo-header-info` - Restore feature by kovsu · Pull Request #8869 · refined-github/refined-github · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@kovsu
Copy link
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
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
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

@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

0