ldraney/palinks
1
0
Fork 0
Code Issues 12 Pull requests 1 Projects Releases Packages Wiki Activity Actions

Add target="_blank" to external links in view/config pages #48

New issue
Closed
opened 2026-06-13 13:18:43 +00:00 by ldraney · 0 comments
ldraney commented 2026-06-13 13:18:43 +00:00
Owner
Copy link

Type

Bug

Lineage

Standalone — discovered during PR #47 review. Originates from PR #44 changes.

Repo

ldraney/palinks

What Broke

External links on the public view and config pages navigate away from palinks instead of opening in a new tab. Users lose their place when clicking any outbound link.

Repro Steps

  1. Visit any palinks public page with external links
  2. Click an external link card
  3. Observe: browser navigates away from palinks entirely

Expected Behavior

External links should open in a new tab via target="_blank" rel="noopener noreferrer", keeping the user on the palinks page.

Environment

  • Cluster/namespace: prod
  • Service version/commit: post-PR #44 merge

Acceptance Criteria

  • All external <a> links in view templates open in new tabs
  • All external <a> links in config templates open in new tabs
  • rel="noopener noreferrer" included for security
  • No regression in internal navigation

Related

  • project-palinks — project this affects
  • ldraney/palinks #44 — PR that introduced the current link behavior
### Type Bug ### Lineage Standalone — discovered during PR #47 review. Originates from PR #44 changes. ### Repo `ldraney/palinks` ### What Broke External links on the public view and config pages navigate away from palinks instead of opening in a new tab. Users lose their place when clicking any outbound link. ### Repro Steps 1. Visit any palinks public page with external links 2. Click an external link card 3. Observe: browser navigates away from palinks entirely ### Expected Behavior External links should open in a new tab via `target="_blank" rel="noopener noreferrer"`, keeping the user on the palinks page. ### Environment - Cluster/namespace: prod - Service version/commit: post-PR #44 merge ### Acceptance Criteria - [ ] All external `<a>` links in view templates open in new tabs - [ ] All external `<a>` links in config templates open in new tabs - [ ] `rel="noopener noreferrer"` included for security - [ ] No regression in internal navigation ### Related - `project-palinks` — project this affects - `ldraney/palinks #44` — PR that introduced the current link behavior
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
40-canvas-view-activity-sizing
28-custom-domain-reverse-proxy
No results found.
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
ldraney/palinks#48
No description provided.