[4.2]Corrected different multi-select behavior in media manage

[rajputanuj31]

Pull Request #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.-.Demo-.Trim.mp4

[dgrammatiko]

@rajputanuj31 I think the expected behaviour involves both ctrl and shift keys, I had started fixing it here: #39679 (comment) (it needs the shift key logic as explained in that comment)

[richard67]

Besides this, the javascript linter at the drone CI tests reports errors here https://ci.joomla.org/joomla/joomla-cms/61584/1/26 :

+ npm run lint:js

> joomla@4.0.0 lint:js
> eslint --config build/.eslintrc --ignore-pattern '/media/' --ext .es6.js,.es6,.vue .

/********/src/administrator/components/com_media/resources/scripts/components/browser/items/item.es6.js
  117:17  error  'event' is defined but never used  no-unused-vars

/********/src/administrator/components/com_media/resources/scripts/components/browser/items/row.vue
  96:13  error  'event' is defined but never used  no-unused-vars

✖ 2 problems (2 errors, 0 warnings)

[rajputanuj31]

@rajputanuj31 I think the expected behaviour involves both ctrl and shift keys, I had started fixing it here: #39679 (comment) (it needs the shift key logic as explained in that comment)

@dgrammatiko Is this looks fine??

Media.-Trim-2.mp4

[dgrammatiko]

@rajputanuj31 please rebase this pr to 4.3-dev

[rajputanuj31]

Please Review this PR #39824.
Previously I have forked only 4.2 branch so for rebasing had to fork again 🙃