? ? Success

User tests: Successful: Unsuccessful:

avatar bahl24
bahl24
18 Feb 2019

Signed-off-by: Nitish Bahl nitishbahl24@gmail.com

Pull Request for Issue #23792 .

Summary of Changes

Removed unused method disconnectedCallback

Testing Instructions

  • Build js files
  • Open the browser console
  • Go to create a new article in the Joomla backend
  • Reduce the browser window width until you see an error in the con

Expected result

No error in console

Actual result

Error in console

Documentation Changes Required

avatar bahl24 bahl24 - open - 18 Feb 2019
avatar bahl24 bahl24 - change - 18 Feb 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Feb 2019
Category JavaScript Repository
avatar dgrammatiko
dgrammatiko - comment - 18 Feb 2019

@bahl24 please close this pr and head to MDN to read the specs

avatar dgrammatiko
dgrammatiko - comment - 18 Feb 2019

Also what you were supposed to fix with this was already patched here:
#23881

avatar bahl24 bahl24 - change - 19 Feb 2019
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2019-02-19 03:43:29
Closed_By bahl24
avatar bahl24 bahl24 - close - 19 Feb 2019
avatar joomla-cms-bot joomla-cms-bot - change - 19 Feb 2019
Category JavaScript Repository JavaScript Repository NPM Change
avatar bahl24 bahl24 - change - 19 Feb 2019
Status Closed New
Closed_Date 2019-02-19 03:43:29
Closed_By bahl24
Labels Added: ?
avatar bahl24 bahl24 - change - 19 Feb 2019
Status New Pending
avatar bahl24 bahl24 - reopen - 19 Feb 2019
avatar bahl24 bahl24 - change - 19 Feb 2019
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2019-02-19 03:43:53
Closed_By bahl24
avatar bahl24 bahl24 - close - 19 Feb 2019
avatar Quy
Quy - comment - 19 Feb 2019

@dgrammatiko Does your PR fix #23792?

avatar dgrammatiko
dgrammatiko - comment - 19 Feb 2019

Yes

Add a Comment

Login with GitHub to post a comment