? NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
3 Feb 2022

Partial pull Request for Issue #36533.

Summary of Changes

Turns the element which shows an image in the media manager from a background div into a propper image element. Full credit for this pr goes to @dgrammatiko as he is the original author in #36550.

Testing Instructions

  • Open the media manager list with the local adapter.
  • Optionally you can also install the FTP adapter from DPmedia to test it with an external adapter.

Actual result BEFORE applying this Pull Request

Image is shown as background image.

Expected result AFTER applying this Pull Request

Image is shown as normal image with loading lazy attribute.

avatar laoneo laoneo - open - 3 Feb 2022
avatar laoneo laoneo - change - 3 Feb 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 Feb 2022
Category Administration com_media NPM Change Repository
avatar Quy
Quy - comment - 4 Feb 2022

In browser's console when displaying an SVG image.

TypeError: this.item.thumb_path is undefined

avatar laoneo laoneo - change - 4 Feb 2022
Labels Added: NPM Resource Changed ?
a45358e 4 Feb 2022 avatar laoneo cs
avatar laoneo
laoneo - comment - 4 Feb 2022

Thanks @Quy , fixed it

05610fd 4 Feb 2022 avatar laoneo cs
avatar Quy
Quy - comment - 4 Feb 2022

The SVG should display by adding src and removing loading, width, and height attributes.

36930-thumbnail
36930-markup

avatar laoneo
laoneo - comment - 4 Feb 2022

All we have in the list is the thumb url, not the full url. But this issue also exists on the 4.2-dev branch. So it needs to be addressed separately. Because there is a general issue how to handle media without a thumbnail.
image

avatar Quy Quy - test_item - 4 Feb 2022 - Tested successfully
b7c2241 7 Feb 2022 avatar laoneo cs
avatar Quy Quy - test_item - 8 Feb 2022 - Tested successfully
avatar Quy
Quy - comment - 8 Feb 2022

I have tested this item successfully on 248ffe6


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

avatar HarshJaiswani HarshJaiswani - test_item - 9 Feb 2022 - Tested successfully
avatar HarshJaiswani
HarshJaiswani - comment - 9 Feb 2022

I have tested this item successfully on 248ffe6


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

avatar Quy Quy - change - 9 Feb 2022
Status Pending Ready to Commit
avatar Quy
Quy - comment - 9 Feb 2022

RTC


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

avatar roland-d roland-d - change - 10 Feb 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-02-10 20:56:02
Closed_By roland-d
Labels Added: ?
avatar roland-d roland-d - close - 10 Feb 2022
avatar roland-d roland-d - merge - 10 Feb 2022
avatar roland-d
roland-d - comment - 10 Feb 2022

Thank you,

Add a Comment

Login with GitHub to post a comment