NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
17 Oct 2019

Pull Request for Issue # .

Summary of Changes

  • Remove a global var, whoever did this is not my friend (hint: probably I did it)
  • Unmute some code that was closing automatically a modal

Note: The Joomla.Modal API is always on the parent window!!!

(eg if there is an iframe and you need to call it from there you must call it like: window.parent.Joomla.Modal.getCurrent().close())

Testing Instructions

Run npm install !important

Go to http://j4.local/administrator/index.php?option=com_finder&view=index
click on the Index button
The modal SHOULD close automatically at the end of the indexing process

Expected result

Actual result

Documentation Changes Required

No, cleanup

avatar dgrammatiko dgrammatiko - open - 17 Oct 2019
avatar dgrammatiko dgrammatiko - change - 17 Oct 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Oct 2019
Category Administration com_finder JavaScript Repository NPM Change
avatar dgrammatiko dgrammatiko - change - 17 Oct 2019
The description was changed
avatar dgrammatiko dgrammatiko - edited - 17 Oct 2019
avatar dgrammatiko dgrammatiko - change - 17 Oct 2019
The description was changed
avatar dgrammatiko dgrammatiko - edited - 17 Oct 2019
avatar Quy
Quy - comment - 17 Oct 2019

The last message flashes quickly as the modal closes. Is this the desired behavior?

avatar dgrammatiko
dgrammatiko - comment - 17 Oct 2019

The last message flashes quickly as the modal closes. Is this the desired behavior?

Might be the globally applied animations, I didn't notice to be honest...

avatar snehaM26 snehaM26 - test_item - 19 Oct 2019 - Tested unsuccessfully
avatar snehaM26
snehaM26 - comment - 19 Oct 2019

I have tested this item 🔴 unsuccessfully on 5fb0a6e

Tested this issue


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

avatar dgrammatiko
dgrammatiko - comment - 19 Oct 2019

@snehaM26 dis you run ‘npm I’?

avatar snehaM26 snehaM26 - test_item - 19 Oct 2019 - Tested unsuccessfully
avatar snehaM26
snehaM26 - comment - 19 Oct 2019

I have tested this item 🔴 unsuccessfully on 5fb0a6e

Tested uncessessfully.


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

avatar jaydevingawale jaydevingawale - test_item - 19 Oct 2019 - Tested unsuccessfully
avatar jaydevingawale
jaydevingawale - comment - 19 Oct 2019

I have tested this item 🔴 unsuccessfully on 5fb0a6e

it is not closing after the indexing is completed but if we hit the window.parent.Joomla.Modal.getCurrent().close() in console it gets closed.


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

avatar dgrammatiko dgrammatiko - change - 19 Oct 2019
The description was changed
avatar dgrammatiko dgrammatiko - edited - 19 Oct 2019
avatar dgrammatiko dgrammatiko - change - 19 Oct 2019
Labels Added: NPM Resource Changed ?
avatar dkmkk dkmkk - test_item - 19 Oct 2019 - Tested successfully
avatar dkmkk
dkmkk - comment - 19 Oct 2019

I have tested this item ✅ successfully on 0c97f8c

I had applied the patch:
I can confirm the expected results.


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

avatar ahghatol ahghatol - test_item - 19 Oct 2019 - Tested successfully
avatar ahghatol
ahghatol - comment - 19 Oct 2019

I have tested this item ✅ successfully on 0c97f8c


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

avatar sanderpotjer sanderpotjer - test_item - 19 Oct 2019 - Tested successfully
avatar sanderpotjer
sanderpotjer - comment - 19 Oct 2019

I have tested this item ✅ successfully on 0c97f8c

Modal closes after index finished.


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

avatar wilsonge
wilsonge - comment - 19 Oct 2019

Merging. But we should keep an eye on the animations I guess

avatar wilsonge wilsonge - change - 19 Oct 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-10-19 15:40:44
Closed_By wilsonge
avatar wilsonge wilsonge - close - 19 Oct 2019
avatar wilsonge wilsonge - merge - 19 Oct 2019

Add a Comment

Login with GitHub to post a comment