?
avatar Bakual
Bakual
8 Jan 2018

Steps to reproduce the issue

Switcher elements as well as alerts don't work in IE11

Expected result

Working CEs

Actual result

image

image

System information (as much as possible)

Internet Explorer 11 on Windows 7 Enterprise

Additional comments

@dgt41 I thought we have a polyfill for browsers who don't support webcomponents (eg IE11)?

avatar Bakual Bakual - open - 8 Jan 2018
avatar joomla-cms-bot joomla-cms-bot - change - 8 Jan 2018
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 8 Jan 2018
avatar franz-wohlkoenig franz-wohlkoenig - change - 8 Jan 2018
Category JavaScript
avatar franz-wohlkoenig franz-wohlkoenig - change - 8 Jan 2018
Build staging 4.0-dev
avatar dgt41
dgt41 - comment - 8 Jan 2018

It's the CustomEvent that needs patching for IE11, the code is simple like 3 lines (I thought I did that already, maybe on the other repo, the Joomla-projects/custom-elelements).

avatar C-Lodder
C-Lodder - comment - 8 Jan 2018

This is what I use for IE11 on my project:

var event = document.createEvent('Event');
event.initEvent('onFooBar', true, false);
window.dispatchEvent(event);

window.addEventListener('onFooBar', function() {
   /// something here
}, false);
avatar franz-wohlkoenig franz-wohlkoenig - change - 20 Jan 2018
Status New Discussion
avatar brianteeman
brianteeman - comment - 23 Mar 2018

@dgt41 has this been fixed?

avatar Bakual
Bakual - comment - 24 Mar 2018

@brianteeman Seems to be. The switcher works now in IE11.
I only get a JS error for the tab-state script (tabs-state.min.js (1,563) object doesn't support "evaluate" or so).

avatar Bakual Bakual - change - 24 Mar 2018
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2018-03-24 06:39:49
Closed_By Bakual
avatar Bakual Bakual - close - 24 Mar 2018
avatar dgt41
dgt41 - comment - 24 Mar 2018

@Bakual about tabs-state.js: this script won't be used in J4 at all since the stageful functionality of tabs is already integrated in our new tabs component (which is fully accessible compared to the foo-bar BS). There is already a PR for showcasing the changes needed in the layouts (just change the

JHtml::_('bootstrap.startTabSet', ...)

// New code:
JHtml::_('uitab.startTabSet', ...)

This is already merged in the template repo, but will be good to also have it merged in the main repo and do the monkey work to change all the JHtml parts in the layouts

PS. The tab-state.js script has already been moved to the media/legacy folder, although I would suggest to just drop it as you already found out that it's not working in IE11

avatar Bakual
Bakual - comment - 24 Mar 2018

I assumed as such since it seems to have a requirement to JQuery and I know you don't like that ?

Add a Comment

Login with GitHub to post a comment