Sitelet https://github.com/processing/p5.js-web-editor/pull/3669
Skip to content

Add rel="noopener noreferrer" to all target="_blank" links - #3669

Merged
raclim merged 3 commits into
processing:developfrom
Jeelislive:fix/noopener-links
Feb 3, 2026
Merged

raclim merged 3 commits into
processing:developfrom
Jeelislive:fix/noopener-links

Conversation

@Jeelislive

Copy link
Copy Markdown
Contributor
  • Prevent tab-napping by pairing target="_blank" with rel="noopener noreferrer".
  • Updated links in:
    - client/modules/IDE/components/AssetListRow.jsx
    - client/modules/IDE/components/Preferences/index.jsx
    - client/modules/IDE/components/show-hint.js
    - client/modules/IDE/components/QuickAddList/QuickAddList.jsx
    - client/modules/About/pages/About.jsx
    - README.md
    - translations/contributor_docs/ko/README.md

No behavior changes; just security hardening.

@Jeelislive

Copy link
Copy Markdown
Contributor Author

@andrewn @peterblazejewicz @cassiano @workergnome review this

@yugalkaushik

Copy link
Copy Markdown
Contributor

@Jeelislive Hi, thank you for the PR and @raclim will be the one reviewing the PRs for this repo.

@raclim raclim left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for your work on this!

For subsequent issues, please make sure to open an issue first before submitting a pull request—more information on this can be found in our Contributor Documentation.

@raclim
raclim merged commit 3010c8b into processing:develop Feb 3, 2026
1 check passed
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.

3 participants