Toolbar buttons on IE 10 throwing JS error???

[ericfernance]

Steps to reproduce the issue

Not sure if this is just me or if others are getting the same thing.

Using the beta of 3.4 from github on a Windows 7 - IE 10 VM. Go to Article Manager and click new.

It should go to a new article but on IE 10 it appears to be throwing a JS error.

Other versions seem unaffected.

Expected result

Expected result would be opening a new article in Article Manager

Actual result

Actual result is a JS error and nothing happens.

System information (as much as possible)

Latest J3.4 download from Github.

Additional comments

[peterlose]

Confirmed

[Fedik]

looks like problem with fireEvent ..
before it was by mootools, and as there no mootools anymore, IE use native one,

just a quick guess ... unfortunately, curently I have no IE for check this theory

[Bakual]

Tracked it down to commit 68b58f3 (PR #4888)

Something for @dgt41 to look at.

[dgrammatiko]

@Bakual from a quick search I think this is more related to core.js
http://stackoverflow.com/questions/17720431/javascript-dispatchevent-is-not-working-in-ie9-and-ie10

I'll try to post a more descriptive answer and maybe a pr later on when I'll get back to my desk

[Bakual]

I think the comment from Fedik is correct. fireEvent previously was provided by MooTools, it then broke because MooTools is no longer there due to your PR.

If you can do a PR to fix this, that would be awesome!

What I find disturbing is that apparently nobody tested creating any item on IE10 since about 2 months, including an alpha release. 😞

[dgrammatiko]

This was the solution mootools used back to 2010:
http://jsbin.com/uvesu3/8/edit?js,output
Here is the talk:
https://mootools.lighthouseapp.com/projects/2706/tickets/976-resolve-fireevent-issues-with-ie9

This is the proposed code:

    if (typeof HTMLElement != 'undefined') {
        HTMLElement.prototype.fireEvent = Element.prototype.fireEvent;
    }

After line 31 in core-uncompressed.js
The only problem I don’t have IE9 to test it 😕
@digitalgarage can you try this code with IE9?

[Fedik]

please test #5914

[jissues-bot]

Set to "closed" on behalf of @zero-24 by The JTracker Application at issues.joomla.org/joomla-cms/5901

[zero-24]

Closing as we have a PR by @Fedik Thanks! #5914

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5901.}

[dgrammatiko]

@Fedik thanks!