Feature RTC NPM Resource Changed PR-5.2-dev Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
22 Jul 2024

Pull Request for Issue # .

Summary of Changes

  • Select the last uploaded file when media manager is inside an iframe (ie, when selecting an image in a media field or inside an editor)

Testing Instructions

  • Create a new article
  • Select an intro image
  • navigate to any folder
  • Upload an image

The image should be selected

Repeat the same with the CMS Content->Media button in tinyMCE editor

Repeat the same with the media button bellow the Codemirror

Actual result BEFORE applying this Pull Request

The file was not selected after upload, user needed to pick the file manually

Expected result AFTER applying this Pull Request

The file is selected after upload, user needed to pick the file manually

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar dgrammatiko dgrammatiko - open - 22 Jul 2024
avatar dgrammatiko dgrammatiko - change - 22 Jul 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Jul 2024
Category JavaScript Administration com_media NPM Change
avatar dgrammatiko dgrammatiko - change - 22 Jul 2024
Labels Added: NPM Resource Changed PR-5.2-dev
avatar Quy
Quy - comment - 22 Jul 2024

Clicking Select or Insert Media button does not insert the image. You have to reselect the image, then the button will work.

avatar dgrammatiko
dgrammatiko - comment - 22 Jul 2024

Clicking Select or Insert Media button does not insert the image. You have to reselect the image, then the button will work.

What's your browser? Seems to work on Webkit (I only tested it there)

avatar Quy
Quy - comment - 22 Jul 2024

Firefox 128.0 and Chrome Version 126.0.6478.128 on Windows

avatar brianteeman
brianteeman - comment - 22 Jul 2024

Confirming the results from @Quy

Chrome Windows - Version 126.0.6478.128 (Official Build) (64-bit)

avatar brianteeman brianteeman - test_item - 22 Jul 2024 - Tested unsuccessfully
avatar brianteeman
brianteeman - comment - 22 Jul 2024

I have tested this item ? unsuccessfully on b3c304d


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

avatar Fedik
Fedik - comment - 22 Jul 2024

@dgrammatiko User selection happen on click, then that "select event" is triggered

avatar dgrammatiko dgrammatiko - change - 22 Jul 2024
Labels Added: Feature
avatar dgrammatiko
dgrammatiko - comment - 22 Jul 2024

User selection happen on click, then that "select event" is triggered

I'm NOT doing an automatic insertion of the uploaded image, I just set the state of the selected file! (means the user still needs to press the insert button, but in this case the file is already selected no need to search for it...) BTW the issue was about SELECTING the image not automatically inserting it, that's a different case

avatar dgrammatiko
dgrammatiko - comment - 22 Jul 2024

@Quy @brianteeman should be ok for all browsers now (tested with Chrome, Webkit, FF)

avatar brianteeman brianteeman - test_item - 22 Jul 2024 - Tested successfully
avatar brianteeman
brianteeman - comment - 22 Jul 2024

I have tested this item ✅ successfully on f669b9f

works as intended now - thanks


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

avatar Quy Quy - test_item - 22 Jul 2024 - Tested successfully
avatar Quy
Quy - comment - 22 Jul 2024

I have tested this item ✅ successfully on f669b9f

Thank you!!!


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

avatar Quy Quy - change - 22 Jul 2024
Status Pending Ready to Commit
avatar Quy
Quy - comment - 22 Jul 2024

RTC


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

avatar dgrammatiko dgrammatiko - change - 22 Jul 2024
Labels Added: RTC
avatar dgrammatiko
dgrammatiko - comment - 22 Jul 2024

Just a note for the person that would merge this PR: please merge first PR #43579 and then this one.

avatar laoneo laoneo - test_item - 26 Jul 2024 - Tested successfully
avatar laoneo
laoneo - comment - 26 Jul 2024

I have tested this item ✅ successfully on d357b4c

Tested image in editor and intro image. Also with the DPMedia restricted plugin.


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

avatar Hackwar
Hackwar - comment - 26 Jul 2024

@Quy @brianteeman could you test this again? There were some changes lately and we are missing one more test.

avatar Quy Quy - test_item - 26 Jul 2024 - Tested successfully
avatar Quy
Quy - comment - 26 Jul 2024

I have tested this item ✅ successfully on 0d98552


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

avatar Quy Quy - alter_testresult - 26 Jul 2024 - laoneo: Tested successfully
avatar Hackwar Hackwar - change - 26 Jul 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-07-26 17:27:55
Closed_By Hackwar
avatar Hackwar Hackwar - close - 26 Jul 2024
avatar Hackwar Hackwar - merge - 26 Jul 2024
avatar Hackwar
Hackwar - comment - 26 Jul 2024

Thank you for your contribution @dgrammatiko! ?

Add a Comment

Login with GitHub to post a comment