? NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
10 May 2021

Removes the range slider as you cant use a range slider to enlarge see #33568 for further details

To test
apply the pr and npm i

go to media manager and confirm the sliders are gone from the resize tab and that you can still resize using the number inputs

avatar brianteeman brianteeman - open - 10 May 2021
avatar brianteeman brianteeman - change - 10 May 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 May 2021
Category Administration Language & Strings JavaScript Repository NPM Change Front End Plugins
avatar ChristineWk ChristineWk - test_item - 10 May 2021 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 10 May 2021

I have tested this item successfully on a234439


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

avatar richard67 richard67 - test_item - 10 May 2021 - Tested successfully
avatar richard67
richard67 - comment - 10 May 2021

I have tested this item successfully on a234439


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

avatar richard67 richard67 - change - 10 May 2021
Status Pending Ready to Commit
Labels Added: ? ? NPM Resource Changed
avatar richard67
richard67 - comment - 10 May 2021

RTC


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

avatar richard67
richard67 - comment - 10 May 2021

@brianteeman These bloody sliders made me completely misunderstand how resizing images in media manager works. For example I wasn't aware of the possibility to enlarge the image by using the number fields because being focused on what the sliders can do. Now I understand and 100% agree with all you said, and am happy when the sliders will have gone.

avatar rdeutz rdeutz - change - 10 May 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-05-10 12:31:31
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 10 May 2021
avatar rdeutz rdeutz - merge - 10 May 2021
avatar brianteeman
brianteeman - comment - 10 May 2021

Less haste more speed - thanks

Add a Comment

Login with GitHub to post a comment