NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
4 Jan 2022

Pull Request for Issue #36549.

Summary of Changes

  • Pass the persmissions from PHP to JS
  • Pass the properties to the icons

Testing Instructions

Check the issus

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Documentation Changes Required

No, bug

@crystalenka

avatar dgrammatiko dgrammatiko - open - 4 Jan 2022
avatar dgrammatiko dgrammatiko - change - 4 Jan 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 4 Jan 2022
Category JavaScript Administration com_media NPM Change
dcefa79 4 Jan 2022 avatar dgrammatiko meh
avatar dgrammatiko dgrammatiko - change - 4 Jan 2022
Labels Added: NPM Resource Changed ?
avatar crystalenka
crystalenka - comment - 4 Jan 2022

Works on images, there's a minor bug on folders where when you hover the triple-dot icon appears, but when you click it, it disappears again.

screen shot 2022-01-04 at 11 38 43


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

avatar dgrammatiko
dgrammatiko - comment - 4 Jan 2022

The problem is the focus management of the drop-down list (the ellipsis button is a dropdown under the skin) because it's kinda hardcoded to at least some of the options. Suddenly the scope of this PR got way bigger...

avatar crystalenka
crystalenka - comment - 4 Jan 2022

Oops. I have a habit of doing that lately. ?

avatar dgrammatiko
dgrammatiko - comment - 4 Jan 2022

Oh no, you didn't do anything wrong. Others should be blamed here and I think my name would be also on the list. Anyways this requires quite some refactoring of the buttons and probably I will end up rewriting the whole list view as it's completely inaccessible. That said it's not a 5 mins job...

a5f8f3c 4 Jan 2022 avatar dgrammatiko oops
avatar dgrammatiko
dgrammatiko - comment - 4 Jan 2022

The whole focus management is completely foobar, that said the code here should do what it was meant to do (hopefully).

avatar Quy Quy - change - 4 Jan 2022
Title
[4.0][bugfix] Media manager icons respect the permissions
[4.1][bugfix] Media manager icons respect the permissions
avatar Quy Quy - edited - 4 Jan 2022
avatar Quy
Quy - comment - 7 Jan 2022

Folders should not have the pencil icon.

avatar richard67 richard67 - change - 1 Feb 2022
Title
[4.1][bugfix] Media manager icons respect the permissions
[4.2][bugfix] Media manager icons respect the permissions
avatar richard67 richard67 - edited - 1 Feb 2022
avatar richard67 richard67 - change - 1 Feb 2022
Labels Added: ?
Removed: ?
avatar dgrammatiko
dgrammatiko - comment - 2 Feb 2022

@laoneo will take over this one

avatar dgrammatiko dgrammatiko - change - 2 Feb 2022
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2022-02-02 20:22:39
Closed_By dgrammatiko
avatar dgrammatiko dgrammatiko - close - 2 Feb 2022

Add a Comment

Login with GitHub to post a comment