fix(web): SiteBanner external link missing target=_blank + rel=noopener

Open Beginner friendly
#2,001 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
Under an hour
Newbie friendliness
85/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Quiet
Tech stack
typescript
Domain
frontend, web-dev

Research direction

Open apps/web/src/components/SiteBanner.tsx and inspect how the optional linkUrl is rendered for the banner CTA. Update that anchor's external-link attributes, then verify the CTA opens in a new tab while preserving the current SPA page.

Written by the indexing model from the issue text.

Description

Summary

apps/web/src/components/SiteBanner.tsx renders the banner's optional linkUrl as a plain anchor without target="_blank" or rel="noopener noreferrer". Clicking the CTA navigates the SPA away from the current page.

Requirements

  • Add target="_blank" and rel="noopener noreferrer" to the banner CTA link in apps/web/src/components/SiteBanner.tsx

Context

Admin-curated URLs so the security risk is bounded, but the UX is wrong — banners typically point at external announcements that should open in a new tab without sacrificing the user's place on the SPA. Surfaced in the Plan 3 review.

Dominant language
HTML
Stars
33
Forks
104
Avg merge
12h 3m
Merged PRs (30d)
10

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from USRSE/usrse.github.io

All issues in USRSE/usrse.github.io

Similar issues

More Web Dev issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.