? Pending

User tests: Successful: Unsuccessful:

avatar Fedik
Fedik
13 Jan 2018

Summary of Changes

Fix Joomla.Event.dispatch to work in IE 11

Testing Instructions

In developer console run Joomla.Event.dispatch('tada');

Expected result

No error

Actual result

Error "object doesn't support this action"

Documentation Changes Required

nope

avatar Fedik Fedik - open - 13 Jan 2018
avatar Fedik Fedik - change - 13 Jan 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 13 Jan 2018
Category JavaScript
avatar wilsonge wilsonge - change - 13 Jan 2018
Labels Added: ?
avatar wilsonge wilsonge - change - 13 Jan 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-01-13 17:01:57
Closed_By wilsonge
avatar wilsonge wilsonge - close - 13 Jan 2018
avatar wilsonge wilsonge - merge - 13 Jan 2018
avatar dgt41
dgt41 - comment - 13 Jan 2018

I would preferred the poly fill way here, as it would provide customEvent for every script (given that core.js is loaded)

Eg https://developer.mozilla.org/en-US/docs/Web/API/CustomEvent/CustomEvent

avatar Fedik
Fedik - comment - 14 Jan 2018

@dgt41 I did not wanted to introduce external dependency for core.js 😉

hm, but it not that huge, maybe we can include in core.js?

Add a Comment

Login with GitHub to post a comment