? Pending

User tests: Successful: Unsuccessful:

avatar C-Lodder
C-Lodder
3 Jul 2020

Summary of Changes

Use rel="noopener" for more links that open in a new window (target="_blank")

Testing Instructions

Code review

avatar C-Lodder C-Lodder - open - 3 Jul 2020
avatar C-Lodder C-Lodder - change - 3 Jul 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 Jul 2020
Category Administration com_templates Front End com_content com_newsfeeds Modules
avatar C-Lodder C-Lodder - change - 3 Jul 2020
The description was changed
avatar C-Lodder C-Lodder - edited - 3 Jul 2020
avatar zero-24
zero-24 - comment - 3 Jul 2020

can you add the noreferrer there too?

avatar C-Lodder
C-Lodder - comment - 3 Jul 2020

Why?

noopener supports all the browsers that J4 supports

avatar zero-24
zero-24 - comment - 3 Jul 2020

my intention is to add rel = noreferrer noopener ;)

avatar zero-24
zero-24 - comment - 3 Jul 2020

but will merge them as it is some we can later come back and add it. Thanks @C-Lodder

avatar zero-24 zero-24 - change - 3 Jul 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-07-03 15:19:20
Closed_By zero-24
Labels Added: ?
avatar zero-24 zero-24 - close - 3 Jul 2020
avatar zero-24 zero-24 - merge - 3 Jul 2020

Add a Comment

Login with GitHub to post a comment