Skip to content

fix: treat different ports as external links - #11091

Merged
vicancy merged 2 commits into
mainfrom
vicancy-fix-external-link-port
Aug 11, 2026
Merged

vicancy merged 2 commits into
mainfrom
vicancy-fix-external-link-port

Conversation

@vicancy

@vicancy vicancy commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Treat URLs on a different port as external in the modern template
  • Add focused tests for matching URLs and differing ports, protocols, and hostnames

Fixes #11023

Include the URL port when classifying modern-template navigation links and cover same-origin and external URL cases.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 17547e68-2ebc-4de5-860e-5c94ff6aa328
@vicancy
vicancy merged commit ce475a0 into main Aug 11, 2026
10 checks passed
@vicancy
vicancy deleted the vicancy-fix-external-link-port branch August 11, 2026 03:49
This was referenced Sep 18, 2026
This was referenced Sep 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Link to different port number marked as active in Modern template

1 participant