NPM Resource Changed ? ? Failure

User tests: Successful: Unsuccessful:

avatar Arpit-24
Arpit-24
26 Mar 2019

Pull Request for Issue #24179 .

Summary of Changes

Replaced custom open and close functions with bootstrap show and hide functions.

Render iframe in modal body, earlier it wasn't

avatar Arpit-24 Arpit-24 - open - 26 Mar 2019
avatar Arpit-24 Arpit-24 - change - 26 Mar 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 26 Mar 2019
Category Administration Templates (admin) JavaScript Repository NPM Change Layout
avatar Arpit-24 Arpit-24 - change - 26 Mar 2019
Labels Added: NPM Resource Changed ?
avatar SharkyKZ
SharkyKZ - comment - 26 Mar 2019

I don't think you're supposed to use jQuery/Bootstrap here.

avatar franz-wohlkoenig franz-wohlkoenig - change - 11 Apr 2019
Category Administration Templates (admin) JavaScript Repository NPM Change Layout Administration JavaScript Layout Repository Templates (admin)
avatar uglyeoin
uglyeoin - comment - 19 Oct 2019

Thanks for this commit @Arpit-24 very much appreciated.

Regardless of whether there is a patch or not I get this JS error. Nothing happens with the media.

TypeError: this.querySelector(...).open is not a functionjoomla-field-media.js:244:45

I would assume it is not related to this PR, but perhaps you can provide more informatoin.

Could you also address @SharkyKZ point, or ask for clarification if you aren't sure.

Again, thanks for your contribution, you are appreciated.

avatar Quy
Quy - comment - 12 Jan 2020

Closing for stated reason and confirmed by #23742.

avatar Quy Quy - change - 12 Jan 2020
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2020-01-12 03:41:18
Closed_By Quy
Labels Added: ?
avatar Quy Quy - close - 12 Jan 2020
avatar joomla-cms-bot joomla-cms-bot - change - 12 Jan 2020
Category Administration Templates (admin) JavaScript Repository Layout Administration Templates (admin) JavaScript Repository NPM Change Layout

Add a Comment

Login with GitHub to post a comment