Skip to content

[4.2] add noopener noreferrer in plg_system_webauthn.ini#38271

Merged
roland-d merged 3 commits intojoomla:4.2-devfrom
tecpromotion:patch-link-PLG_SYSTEM_WEBAUTHN
Jul 15, 2022
Merged

[4.2] add noopener noreferrer in plg_system_webauthn.ini#38271
roland-d merged 3 commits intojoomla:4.2-devfrom
tecpromotion:patch-link-PLG_SYSTEM_WEBAUTHN

Conversation

@tecpromotion
Copy link
Copy Markdown
Contributor

Summary of Changes

add rel="noopener noreferrer" to external link

@brianteeman I have replaced the ' with ", as it happens more often and there should not be two spellings, at least within an .ini file.

Testing Instructions

code review

Actual result BEFORE applying this Pull Request

no rel

Expected result AFTER applying this Pull Request

rel=

Documentation Changes Required

.

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-4.2-dev labels Jul 13, 2022
@tecpromotion tecpromotion changed the title add noopener noreferrer [4.2] add noopener noreferrer in plg_system_webauthn.ini Jul 13, 2022
@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jul 13, 2022

I have tested this item ✅ successfully on 4ac9242


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38271.

1 similar comment
@brianteeman
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on 4ac9242


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38271.

@Quy Quy changed the title [4.2] add noopener noreferrer in plg_system_webauthn.ini [4.2] add noopener noreferrer in plg_system_webauthn.ini Jul 13, 2022
@Quy Quy removed the Language Change This is for Translators label Jul 13, 2022
@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jul 13, 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38271.

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators RTC This Pull Request is Ready To Commit labels Jul 13, 2022
@roland-d roland-d merged commit 53d85db into joomla:4.2-dev Jul 15, 2022
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jul 15, 2022
@roland-d
Copy link
Copy Markdown
Contributor

Thank you

@roland-d roland-d added this to the Joomla! 4.2.0 milestone Jul 15, 2022
@tecpromotion tecpromotion deleted the patch-link-PLG_SYSTEM_WEBAUTHN branch July 15, 2022 20:20
Kostelano added a commit to JPathRu/localisation that referenced this pull request Jul 19, 2022
joomla/joomla-cms#36749 +
joomla/joomla-cms#38228 +
joomla/joomla-cms#38227 - (только для en-GB, у нас исправлено ранее)
joomla/joomla-cms#38255 +
joomla/joomla-cms#38244 +
joomla/joomla-cms#38271 - (только для en-GB, у нас исправлено ранее)
joomla/joomla-cms#38287 - (только для en-GB, у нас исправлено ранее)
joomla/joomla-cms#38301 - (только для en-GB, у нас исправлено ранее)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language Change This is for Translators

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants