-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[plg_system_sef] Add sef URI when using srcset attribute #10464
Conversation
@@ -107,7 +107,7 @@ public function onAfterRender() | |||
|
|||
// Check for all unknown protocals (a protocol must contain at least one alpahnumeric character followed by a ":"). | |||
$protocols = '[a-zA-Z0-9\-]+:'; | |||
$attributes = array('href=', 'src=', 'poster='); | |||
$attributes = array('href=', 'src=', 'srcset=', 'poster='); | |||
foreach ($attributes as $attribute) | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a clean line bevor the foreach?
@zero-24 this PR is not for code style of the SEF plugin ... |
I have tested this item ✅ successfully on 8474310 This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10464. |
I have tested this item ✅ successfully on 8474310 This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10464. |
RTC. Thanks This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10464. |
for 3.6.0 I guess. |
Thanks everybody |
Pull Request for Issue #10463.
Summary of Changes
See #10463
Testing Instructions
/images/sampledata/parks/banner_cradle.jpg
or/<sitepath>/images/sampledata/parks/banner_cradle.jpg
)Please note this will not solve all
srcset
possibilities, only for cases like the one described in the issue (path to one image).