?
avatar ufuk-avcu
ufuk-avcu
9 Sep 2017

Steps to reproduce the issue

Install Joomla 3.7.5
Install Yootheme Pro 1.9.6
Create with the Pagebuilder a Element, that contains images
Set the width and height for the images in the element

Expected result

With Joomla 3.7.5 the images are shown

Actual result

After an Upgrade to Joomla 3.8.0 RC1 the images not shown

System information (as much as possible)

PHP 7.1
MySql 5.7
Before Upgrade: Joomla 3.7.5 Stable
Joomla 3.8 RC1

Additional comments

Posted this also here: http://yootheme.com/support/question/114582

I tested Joomla 3.8 RC1 + Yootheme Pro 1.9.5 and if you set the width or height for images (not for Background images), the images are not shown anymore. If you use Joomla 3.7.5, there is no Problem.

The Problem is here:

This is ok: src="/xxx/templates/yootheme/cache/auto-crash-18d5e554.png"

But the srcset are broken: srcset="//xxx/xxx/templates/yootheme/cache/auto-crash-18d5e554.png

The //xxx doesn't belong there.

avatar ufuk-avcu ufuk-avcu - open - 9 Sep 2017
avatar joomla-cms-bot joomla-cms-bot - change - 9 Sep 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 9 Sep 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 10 Sep 2017
Category Plugins
avatar C-Lodder
C-Lodder - comment - 10 Sep 2017

They've replied:

Yootheme will release a Joomla 3.8 compatible version after it has been released.

avatar franz-wohlkoenig franz-wohlkoenig - change - 10 Sep 2017
Status New Information Required
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 10 Sep 2017

@ufuk-avcu cause Comment above can you please close Issue?


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

avatar ufuk-avcu
ufuk-avcu - comment - 10 Sep 2017

Hey. I just thought that this could be relevant from BC's point of view if the behavior changes in other extensions as well.

avatar franz-wohlkoenig franz-wohlkoenig - change - 10 Sep 2017
Status Information Required Discussion
avatar mbabker
mbabker - comment - 10 Sep 2017

If it can be isolated what specifically changed, we can look into the issue. The report lacks any detail to be able to do anything.

Core doesn't do anything to generate srcset attributes by default. #15300 added support for processing srcset in the system SEF plugin in full, so maybe that change affected the Yootheme handling. But that is IMO hardly a B/C break.

avatar Oudmane
Oudmane - comment - 10 Sep 2017

@yootheme fixed this at pro v1.9.6

avatar ufuk-avcu
ufuk-avcu - comment - 12 Sep 2017

@Oudmane tried now with Yootheme Pro 1.9.6 and it's not fixed.

avatar wronax
wronax - comment - 14 Sep 2017

I can confirm that this is a bug in the core joomla SEF plugin implementation. The same behaviour can be noticed in DJ-MediaTools gallery which also uses srcset attribute. The relative site path is added to image url no matter if image url contains it already what creates doubled relative path at the beginning of the image url.


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

avatar tonypartridge
tonypartridge - comment - 18 Sep 2017

Can we close this? Pull is in: #17978

avatar joomla-cms-bot joomla-cms-bot - close - 19 Sep 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 19 Sep 2017
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2017-09-19 05:09:58
Closed_By franz-wohlkoenig
avatar joomla-cms-bot
joomla-cms-bot - comment - 19 Sep 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 19 Sep 2017

closed as having Pull Request #17978


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

Add a Comment

Login with GitHub to post a comment