Added rel="noopener noreferrer" to links with target="_blank" (P3: modules/widgets/)#9643
Added rel="noopener noreferrer" to links with target="_blank" (P3: modules/widgets/)#9643stevenlinx wants to merge 1 commit intoAutomattic:masterfrom
Conversation
|
@stevenlin-x thank you for your work on this... We're currently doing some file juggling that will be over in some days. I need to relabel this PR to "Needs author reply" because we really need testing instructions for each of the changes or they take us hours to reproduce a functional testing flow... I'll be watching these PRs in case you want to add the testing instructions. Otherwise It's gonna take me some time to write them so they get properly reviewed. |
|
This PR has been marked as stale. This happened because:
No further action is needed. But it's worth checking if this PR has clear testing instructions, is it up to date with master, and it is still valid. Feel free to close this issue if you think it's not valid anymore — if you do, please add a brief explanation. |
|
This PR has been marked as stale. This happened because:
No further action is needed. But it's worth checking if this PR has clear testing instructions, is it up to date with master, and it is still valid. Feel free to close this issue if you think it's not valid anymore — if you do, please add a brief explanation. |
|
This PR has been marked as stale. This happened because:
No further action is needed. But it's worth checking if this PR has clear testing instructions, is it up to date with master, and it is still valid. Feel free to close this issue if you think it's not valid anymore — if you do, please add a brief explanation. |
|
why was this pull request never completed? |
|
@Zohaib321 I'm afraid the author never got the chance to come back to add more information to this Pull Request, as per this comment: If you'd like to give it a try, do not hesitate! |
Fixes #8128