Skip to content

same-page-links - Support PR links on Branches list#9360

Merged
fregante merged 3 commits into
mainfrom
copilot/fix-show-associated-branch-prs
May 7, 2026
Merged

same-page-links - Support PR links on Branches list#9360
fregante merged 3 commits into
mainfrom
copilot/fix-show-associated-branch-prs

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 6, 2026

  • Reproduce current baseline checks and note existing unrelated failures
  • Inspect show-associated-branch-prs-on-fork and same-page-links implementations
  • Move handling of the native branch PR link target behavior to same-page-links
  • Update show-associated-branch-prs-on-fork to stop setting _blank directly
  • Add/adjust focused tests if available for this behavior
  • Run targeted lint/tests for changed files
  • Manually verify on branches page and capture UI screenshot

@fregante fregante changed the title [WIP] Fix target blank for associated branch PR links same-page-links - Support PR links on Branches list May 7, 2026
Copy link
Copy Markdown
Member

@fregante fregante left a comment

Choose a reason for hiding this comment

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

whoah, it worked. All of these open locally. The middle one is native

Image

@fregante fregante marked this pull request as ready for review May 7, 2026 18:20
@fregante fregante enabled auto-merge (squash) May 7, 2026 18:20
@fregante fregante merged commit 243f731 into main May 7, 2026
9 checks passed
@SunsetTechuila SunsetTechuila deleted the copilot/fix-show-associated-branch-prs branch May 7, 2026 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

show-associated-branch-prs-on-fork + same-page-link drop target blank

2 participants