NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
2 Oct 2020

PR for #29332

Steps to reproduce the issue

Got to media manager of a folder
Make sure the information block is displayed by clickin on the i icon in the toolbar
use the zoom out button as far as you can zoom out
Hover over an image or folder

Expected result

nice display
image

Actual result

When image list zoomed out to maximum zoom there is not enough room for both squares
image

Summary of changes

This PR removes the smallest zoom level

Test instructions

Don't forget to run npm ci

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
4.00

avatar brianteeman brianteeman - open - 2 Oct 2020
avatar brianteeman brianteeman - change - 2 Oct 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 Oct 2020
Category Administration com_media NPM Change JavaScript
avatar brianteeman brianteeman - change - 2 Oct 2020
Labels Added: NPM Resource Changed ?
avatar joomla-cms-bot joomla-cms-bot - change - 2 Oct 2020
Category Administration com_media NPM Change JavaScript Administration com_media NPM Change JavaScript Unit Tests
avatar blueforce blueforce - test_item - 17 Oct 2020 - Tested successfully
avatar blueforce
blueforce - comment - 17 Oct 2020

I have tested this item successfully on 70287ed


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

avatar brianteeman brianteeman - change - 22 Oct 2020
Labels Added: Conflicting Files ?
avatar joomla-cms-bot joomla-cms-bot - change - 22 Oct 2020
Category Administration com_media NPM Change JavaScript Unit Tests Administration com_media NPM Change Unit Tests
avatar brianteeman brianteeman - change - 22 Oct 2020
Labels Removed: Conflicting Files
avatar joomla-cms-bot joomla-cms-bot - change - 22 Oct 2020
Category Administration com_media NPM Change Unit Tests Administration com_media NPM Change JavaScript Unit Tests
avatar brianteeman
brianteeman - comment - 22 Oct 2020

Conflicts resolved

avatar N6REJ
N6REJ - comment - 3 Nov 2020

close this please.. we already have pr #31140

avatar brianteeman
brianteeman - comment - 3 Nov 2020

this pr is nothing to do with that pr for mobile views

avatar ceford
ceford - comment - 18 Nov 2020

Elimination of the smallest size helps a little. I was thinking that resizing might apply to images only, leaving folders a constant size. But to be honest the media manager layout is such a mess on small screens I think it would be best to sort that out before applying this PR.

This is a worst case example:

screen shot 2020-11-18 at 18 34 18


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

avatar adj9 adj9 - test_item - 24 Nov 2020 - Tested successfully
avatar adj9
adj9 - comment - 24 Nov 2020

I have tested this item successfully on 912925f

I installed the PR and the features works without any problems.

While doing some tests, I saw that there is always overlap between the buttons. Is it possible to stop a bit earlier?
A zoom level I think is enough.


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

avatar gostn gostn - test_item - 24 Nov 2020 - Tested successfully
avatar gostn
gostn - comment - 24 Nov 2020

I have tested this item successfully on 912925f


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

avatar alikon alikon - change - 24 Nov 2020
Status Pending Ready to Commit
avatar alikon
alikon - comment - 24 Nov 2020

RTC


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

avatar laoneo laoneo - change - 30 Nov 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-11-30 14:41:10
Closed_By laoneo
Labels Added: ?
Removed: ?
avatar laoneo laoneo - close - 30 Nov 2020
avatar laoneo laoneo - merge - 30 Nov 2020
avatar laoneo
laoneo - comment - 30 Nov 2020

Thanks!

avatar brianteeman
brianteeman - comment - 30 Nov 2020

Thank you

Add a Comment

Login with GitHub to post a comment