Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not Displaying in IE 11 - Even the demo page #212

Closed
itssampath opened this issue Aug 29, 2019 · 4 comments
Closed

Not Displaying in IE 11 - Even the demo page #212

itssampath opened this issue Aug 29, 2019 · 4 comments
Assignees
Labels
💥 Bug Something isn't working

Comments

@itssampath
Copy link

Simple-keyboard version
We tried to use Simple keyboard in Angular 8. Working fine in FireFox.

Describe the bug
Not Displayed in IE 11. I could see script error.
Root cause is - In index.js
(l = b.classList).add.apply(l, _toConsumableArray(t.getButtonThemeClasses(e)))
throws "Argument not optional"

Screenshots
Try opening demo page in IE 11. No Keyboard is displayed.

@hodgef hodgef added the ⛔️ Won't Fix This will not be worked on for reasons provided in the ticket label Aug 29, 2019
@hodgef
Copy link
Owner

hodgef commented Aug 29, 2019

Hello @itssampath,

The simple-keyboard demos are coded in ES6+, thus not supported in Internet Explorer.

For example:

let keyboard = new Keyboard({
  onChange: input => onChange(input),
  onKeyPress: button => onKeyPress(button)
});

Will not run in IE due to the arrow functions.

In a previous demo I had the following message shown:

image

I will look into adding it back just as a heads up to users.

Regards,
Francisco Hodge

@nicoabie
Copy link

I completely support the decision of the author @hodgef to not support IE11.
Unfortunately there exist cases where people are hired to make it work in IE11 because some institutions like banks still work with such browsers.

The issue here described is because IE11 does not support certain operations with multiple arguments in classList see here

In order to fix this a classList polyfill is required.

I suggest this one which has the latest version of https://github.com/eligrey/classList.js. Be aware that most documentation will tell you to run npm i --save classlist.js but that one does NOT resolve this issue because it is outdated.

I leave this comment in case anyone has to make it work in IE11

@hodgef
Copy link
Owner

hodgef commented Apr 20, 2020

Thanks for sharing this fix @nicoabie ! Much appreciated

@hodgef
Copy link
Owner

hodgef commented Mar 21, 2021

Update: simple-keyboard v3+ now supports IE11 😄

Compatibility list: https://github.com/hodgef/simple-keyboard#-compatibility

Please note : The demos (meant for devs) do not support IE, but the end product (the library) does.

Regards,
Francisco Hodge

@hodgef hodgef added 💥 Bug Something isn't working and removed ⛔️ Won't Fix This will not be worked on for reasons provided in the ticket labels Mar 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💥 Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants