Skip to content

clean-repo-sidebar - Apply to mobile#9618

Merged
fregante merged 3 commits into
refined-github:mainfrom
yangbun-GIT:fix/9597-clean-repo-sidebar-mobile
May 26, 2026
Merged

clean-repo-sidebar - Apply to mobile#9618
fregante merged 3 commits into
refined-github:mainfrom
yangbun-GIT:fix/9597-clean-repo-sidebar-mobile

Conversation

@yangbun-GIT

Copy link
Copy Markdown
Contributor

Copilot AI review requested due to automatic review settings May 25, 2026 19:34
@github-actions github-actions Bot added the bug label May 25, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adjusts the “clean repo sidebar” styling by changing when a redundant links row is hidden and keeping desktop-only alignment tweaks in a media query.

Changes:

  • Moves the “hide redundant releases/deployments/contributors links” rule to apply at all viewport widths.
  • Keeps the icon alignment tweak scoped to @media (width >= 768px) but shifts the media block below the new global rule.

Comment thread source/features/clean-repo-sidebar.css Outdated
Comment thread source/features/clean-repo-sidebar.css Outdated

@fregante fregante left a comment

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.

Finally a PR with screenshots 🙏

@fregante

Copy link
Copy Markdown
Member

Will wait for @SunsetTechuila to review since he opened the issue

@SunsetTechuila SunsetTechuila left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The correct fix is to delete the line you moved from the media query and update this one instead:

.BorderGrid-cell > .mt-3:not(:has(> a[href$='releases/new'])) {

.mt-3 should be changed to :is(.tmp-mt-3, .mt-3)

@yangbun-GIT yangbun-GIT force-pushed the fix/9597-clean-repo-sidebar-mobile branch from 41e2fd3 to cd67cd8 Compare May 26, 2026 08:02
@github-actions

Copy link
Copy Markdown

Please avoid force-pushing to this PR. It makes it harder for reviewers to track what changed since the last review.

Just push new commits on top of the branch instead — PRs are squashed on merge, so the commit history doesn't need to be clean.

@yangbun-GIT

Copy link
Copy Markdown
Contributor Author

Updated the selector as suggested. The latest check passed as well.

Noted on the force-push guidance. I’ll push follow-up commits normally from now on.

// Hide the whole section if there's no releases
!elementExists('.octicon-tag', releasesSection)
// Don't hide the section if it has a "Create new release" link
&& !elementExists('a[href$="releases/new"]', releasesSection)

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.

🙌

@fregante fregante merged commit f496c2d into refined-github:main May 26, 2026
13 checks passed
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.

clean-repo-sidebar - Apply to mobile

4 participants