[#5901] - Toolbar buttons on IE 10 throwing JS error???
- Closed
- 28 Jan 2015
- Urgent
- Build: master
- # 5901
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
brianteeman
- | Status | New | ⇒ | Confirmed |
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
| Priority | Medium | ⇒ | Urgent |
@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
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. 
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
| Labels |
Added:
?
|
||
| Status | Confirmed | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-01-28 17:22:47 |
| Closed_By | ⇒ | 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.
Set to "closed" on behalf of @zero-24 by The JTracker Application at issues.joomla.org/joomla-cms/5901
| Closed_Date | 2015-01-28 17:22:47 | ⇒ | 2015-01-28 17:22:46 |
Confirmed