Skip to content
This repository has been archived by the owner on May 15, 2024. It is now read-only.

Added a target property #354

Merged
merged 23 commits into from Apr 1, 2020
Merged

Added a target property #354

merged 23 commits into from Apr 1, 2020

Conversation

olawanlejoel
Copy link
Contributor

@olawanlejoel olawanlejoel commented Mar 11, 2020

I discovered when the "Contribute on GitHub" button is clicked, it just loads closing up the file the individual was actually reading which is not suppose to be. I added a target property so it opens up a new tab whenever it is clicked, in that case the user can still continue what he/she was doing previously before clicking the button so he/she does not loose track of where he/she was.

I discovered when the "Contribute on GitHub button is clicked, it just loads closing up the file the individual was actually reading which is not suppose to be. I added a target property so it opens up a new tab whenever it is clicked, in that case the user can still continue what he/she was doing previously before clicking the button so he/she does not loose track of where he/she was.
Copy link
Collaborator

@Volker-E Volker-E left a comment

Choose a reason for hiding this comment

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

This should, if agreed on, done for all pages in the style guide, not just index.html

@olawanlejoel
Copy link
Contributor Author

Can I work on implementing it on all pages?

@Volker-E
Copy link
Collaborator

@olawanlejoel For sure, happily do so! Please make sure, that you also add rel="noopener" alongside target="_blank". See https://mathiasbynens.github.io/rel-noopener/ for more information on the specifics.

@olawanlejoel
Copy link
Contributor Author

Thanks @Volker-E.

@olawanlejoel
Copy link
Contributor Author

olawanlejoel commented Mar 12, 2020

@Volker-E , I just implemented the target="_blank" and rel="noopener" on all the required pages.

@olawanlejoel
Copy link
Contributor Author

I have reviewed and i think that should be all the pages that require this changes

@Volker-E Volker-E merged commit dec2471 into wikimedia:master Apr 1, 2020
@Volker-E
Copy link
Collaborator

Volker-E commented Apr 1, 2020

Thanks @olawanlejoel!

@olawanlejoel
Copy link
Contributor Author

olawanlejoel commented Apr 1, 2020 via email

@Volker-E
Copy link
Collaborator

Volker-E commented Apr 1, 2020

We've decided against it for design and branding reasons. We wanted to emphasize the Wikimedia branding while providing an easy discoverable GitHub link.

@olawanlejoel
Copy link
Contributor Author

olawanlejoel commented Apr 2, 2020 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants