? Release Blocker NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar Fedik
Fedik
4 Oct 2022

Pull Request for Issue #37054 .

Summary of Changes

Update our jdragndrop plugin to work only with "Files".

Testing Instructions

Run npm install, and follow #37054
Would be nice to test in all Browsers.

Actual result BEFORE applying this Pull Request

Text drag is broken
Image upload works

Expected result AFTER applying this Pull Request

Text drag works
Image upload works

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 Fedik Fedik - open - 4 Oct 2022
avatar Fedik Fedik - change - 4 Oct 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 4 Oct 2022
Category JavaScript Repository NPM Change
avatar Fedik Fedik - change - 4 Oct 2022
Labels Added: NPM Resource Changed ?
avatar richard67
richard67 - comment - 5 Oct 2022

Added release blocker label as inherited from the issue.

avatar HLeithner
HLeithner - comment - 6 Oct 2022

I have tested this item successfully on 9f521c2


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

avatar HLeithner HLeithner - test_item - 6 Oct 2022 - Tested successfully
avatar viocassel
viocassel - comment - 6 Oct 2022

I have tested this item successfully on 9f521c2


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

avatar viocassel viocassel - test_item - 6 Oct 2022 - Tested successfully
avatar alikon alikon - change - 6 Oct 2022
Status Pending Ready to Commit
avatar alikon
alikon - comment - 6 Oct 2022

RTC


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

avatar roland-d roland-d - change - 7 Oct 2022
Labels Added: ? Release Blocker
avatar roland-d
roland-d - comment - 7 Oct 2022

@Fedik After checking out this branch, running npm ci and testing the editor, the drag and drop of text works but drag and drop of images no longer works. I do not get the dotted line around the block nor does anything happen when I drop the image on the editor.

This is running Firefox 105.0.2 on Linux.

avatar Fedik
Fedik - comment - 7 Oct 2022

Thanks for checking that. I have fixed Firefox. Please test again

avatar N6REJ
N6REJ - comment - 8 Oct 2022

I have tested this item ? unsuccessfully on c683ae2

Works in firefox but not in chrome
image


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38900.
avatar N6REJ N6REJ - test_item - 8 Oct 2022 - Tested unsuccessfully
avatar Fedik Fedik - change - 8 Oct 2022
Status Ready to Commit Pending
avatar Fedik
Fedik - comment - 8 Oct 2022

I set it back to Pending

avatar Fedik
Fedik - comment - 8 Oct 2022

@N6REJ are you sure it not a browser cache?
For me works in Chrome and Firfox

avatar roland-d
roland-d - comment - 9 Oct 2022

I have tested this item successfully on c683ae2

Before applying the patch I cannot select and move the text. After applying the patch I am able to upload a image via drag n drop and also text can be selected and moved.


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

avatar roland-d roland-d - test_item - 9 Oct 2022 - Tested successfully
avatar N6REJ
N6REJ - comment - 9 Oct 2022

I have tested this item successfully on c683ae2

@Fedik Must have been. works fine today.


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

avatar N6REJ N6REJ - test_item - 9 Oct 2022 - Tested successfully
avatar N6REJ N6REJ - change - 9 Oct 2022
Status Pending Ready to Commit
avatar N6REJ
N6REJ - comment - 9 Oct 2022

RTC

avatar HLeithner HLeithner - change - 15 Oct 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-10-15 08:14:21
Closed_By HLeithner
avatar HLeithner HLeithner - close - 15 Oct 2022
avatar HLeithner HLeithner - merge - 15 Oct 2022
avatar HLeithner
HLeithner - comment - 15 Oct 2022

Thanks

Add a Comment

Login with GitHub to post a comment