[5.1] Corrected different multi-select behavior in media manager.

[rajputanuj31]

Pull Request for issue #33637
Summary of Changes
Multi select without using shift key

Testing Instructions

Open Media
Try to Select media manager files.

Actual result BEFORE applying this Pull Request

You can not select multiple items without holding the shift key.

Expected result AFTER applying this Pull Request

You can select multiple items by holding ctrl key & clicking on items.
You can also select all files between two selected files by holding shift key.

Media.-Trim-2.mp4

[richard67]

@rajputanuj31 When you create a PR, why do you delete stuff which is prepared in the description, like e.g. the "Pull request for issue #..." at the top, or the section about documentation, which requires the right check boxes to be checked? We do not have that stuff for nothing in our pull request description template. So please add the Pull Request for issue #33637 to the top so we know for which issue the pull request is made, and add back the documentation section and check the right check boxes. Thanks in advance.

[brianteeman]

There are unrelated files in this pull request :(

[rajputanuj31]

There are unrelated files in this pull request :(

Fixed.

[dgrammatiko]

@rajputanuj31 can you check if the rtl languages need any adjustment? Other than that this seems ready for some testing

[rajputanuj31]

@rajputanuj31 can you check if the rtl languages need any adjustment? Other than that this seems ready for some testing

it's working fine.
@dgrammatiko could you please test this PR.

[laoneo]

Please add some inline comments, so we can follow your code. Actually it is hard to understand what exactly is going on with all these if else conditions. I would also recommend that you do early return instead of so many else statements.

[brianteeman]

I have tested this and it works perfectly apart from on the first page which is very weird.

In this video I am shift clicking on the last image and then on the image before it and you will see that a third image is also selected. I can't replicate that on any other screen.

[rajputanuj31]

I have tested this and it works perfectly apart from on the first page which is very weird.

In this video I am shift clicking on the last image and then on the image before it and you will see that a third image is also selected. I can't replicate that on any other screen.

@brianteeman It's working perfectly in my local machine. what should I do now?
@dgrammatiko @laoneo can you please confirm this issue.

[brianteeman]

do you have exactly the same images as I have?

[rajputanuj31]

do you have exactly the same images as I have?

Yes I do have the same images.

[brianteeman]

do you have exactly the same images as I have?

Yes I do have the same images.

all three?

[rajputanuj31]

do you have exactly the same images as I have?

Yes I do have the same images.

all three?

Yes now I am also facing the same issue. I am trying to debug it.

[rajputanuj31]

@dgrammatiko @laoneo @brianteeman please test the PR.

[rajputanuj31]

Hey @laoneo @dgrammatiko @brianteeman @Hackwar I think this PR is ready so please test it. If PR is not ready please leave a comment.

[ceford]

I have tested this item 🔴 unsuccessfully on f971327

There is something wrong here. I applied the patch and ran npm ci (and even npm install). I am using Firefox on a Mac with 5.0.0-beta2-dev and I see no difference in the behaviour with the patch installed and hard page reload. The Ctrl key has no effect. I checked the code to see that all the patches were applied. On Mac/Firefox the Ctrl key brings up a browser dialog (Back, Reload, etc). I could not find a combination of keys that select/deselect an image that does not deselect previously selected images.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39824.}

[HLeithner]

@rajputanuj31 can you look at the response by @ceford please?

[rajputanuj31]

@rajputanuj31 can you look at the response by @ceford please?

Its working perfectly in my local machine.
Screencast from 05-10-23 06:23:07 PM IST.webm

[dgrammatiko]

Mac ctrl (in this context) is the apple button (command)

[webmasterab]

I tested it and it didn't pass.
See video.
Windows PC
Chrome latest version
CTRL pressed

patchtest.mp4