? NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar toroworx
toroworx
15 Mar 2022

Pull Request for Issue #
#37277

Summary of Changes

When using the modal preview for images with large images, images gets cut off.

Testing Instructions

  • Upload a width-wise large image to your Joomla images folder.
  • Go to Joomla media manager and select said image.
  • Preview the image; it's cut off.

Actual result BEFORE applying this Pull Request

Images only partly visible.

Expected result AFTER applying this Pull Request

Images fully visible, because of proper width.

Documentation Changes Required

avatar toroworx toroworx - open - 15 Mar 2022
avatar toroworx toroworx - change - 15 Mar 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Mar 2022
Category Repository NPM Change
avatar toroworx toroworx - change - 15 Mar 2022
The description was changed
avatar toroworx toroworx - edited - 15 Mar 2022
avatar toroworx toroworx - change - 15 Mar 2022
The description was changed
avatar toroworx toroworx - edited - 15 Mar 2022
avatar RickR2H
RickR2H - comment - 15 Mar 2022

@richard67 do you haven an idea why the build is failing?

avatar richard67
richard67 - comment - 15 Mar 2022

@richard67 do you haven an idea why the build is failing?

Because it can? ?

avatar chmst
chmst - comment - 16 Mar 2022

The change was made here #34634.
@dgrammatiko could you have a look?

avatar dgrammatiko
dgrammatiko - comment - 16 Mar 2022

It's fine, I have no clue why it was changed to 200%

avatar Quy
Quy - comment - 16 Mar 2022

I have tested this item successfully on cf75216


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

avatar Quy Quy - test_item - 16 Mar 2022 - Tested successfully
avatar chmst
chmst - comment - 16 Mar 2022

I have tested this item successfully on cf75216


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

avatar chmst chmst - test_item - 16 Mar 2022 - Tested successfully
avatar chmst chmst - change - 16 Mar 2022
Status Pending Ready to Commit
avatar chmst
chmst - comment - 16 Mar 2022

RTC


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

avatar bembelimen bembelimen - change - 17 Mar 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-03-17 20:05:21
Closed_By bembelimen
Labels Added: ? NPM Resource Changed ?
avatar bembelimen bembelimen - close - 17 Mar 2022
avatar bembelimen bembelimen - merge - 17 Mar 2022
avatar bembelimen
bembelimen - comment - 17 Mar 2022

Thx

Add a Comment

Login with GitHub to post a comment