Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#501] Open external links safely in new tab #606

Closed
wants to merge 26 commits into from

Conversation

MasihTak
Copy link
Contributor

Description

As already mentioned in #ISSUE_NUMBER, this PR tackles:
Open external links in new tab with security to prevent tabnapping and other phishing attacks.
This PR does not cover 100% of external links! I need to spend more time fixing others including the documentation links.

Checklist

  • I followed the indications in the CONTRIBUTING
  • The documentation related to the proposed change has been updated accordingly (also comments in code).
  • Have you written new tests for your core changes, as applicable?
  • Have you successfully ran tests with your changes locally?
  • Ready for review! 🚀

Fixes

MasihTak and others added 23 commits October 8, 2019 13:58
Solution based on search result style.
Closes italia#271
Support for SVG logo for GitHub
* Add check on enabling platforms
Expand the interactive area in search results.
Bumps [parallel](https://github.com/grosser/parallel) from 1.17.0 to 1.19.0.
- [Release notes](https://github.com/grosser/parallel/releases)
- [Commits](grosser/parallel@v1.17.0...v1.19.0)

Signed-off-by: dependabot-preview[bot] <[email protected]>
…-1.19.0

Bump parallel from 1.17.0 to 1.19.0
@bfabio
Copy link
Member

bfabio commented Aug 19, 2020

Hello @MasihTak, can you rebase this one on master please?

@sebbalex
Copy link
Member

Hello @MasihTak, can you rebase this one on master please?

Or if you need assistance feel free to ask for it :)

@MasihTak
Copy link
Contributor Author

Hello @bfabio I pushed changes with the Atom and it mess up

@bfabio
Copy link
Member

bfabio commented Aug 20, 2020

@MasihTak Are you on our slack? Let's get in touch there

Copy link
Member

@bfabio bfabio left a comment

Choose a reason for hiding this comment

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

I'm not sure we want noreferrer for .italia.it sites. @libremente WDYT?

@libremente libremente self-requested a review September 4, 2020 14:13
@libremente
Copy link
Member

I'm not sure we want noreferrer for .italia.it sites. @libremente WDYT?

Yes, we don't want it. Could you please correct it @MasihTak? Thanks!

@astagi astagi closed this Mar 4, 2021
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.

Open external links in new tab/window
7 participants