IE11 not working with latest release
See original GitHub issueThe issue here is that a new feature was added in v6 that does not include ie11 supported code. The culprit is the because the userOptions is being spread onto an object. https://github.com/focus-trap/focus-trap/blob/master/index.js#L48
Introduced in https://github.com/focus-trap/focus-trap/pull/125
Does this code get transpiled or should the src be compliant? I’d be happy to make a fix to it, just want to verify what the intention is!
Error is Expected identifier, string or number
@ this line
Steps to Recreate
- Update to v6
- Run in ie11 and see js errors
Issue Analytics
- State:
- Created 3 years ago
- Comments:22 (17 by maintainers)
Top Results From Across the Web
Problems after installing Internet Explorer 11 - Microsoft Learn
Possible solutions to the problems you might encounter after installing IE11, such as crashing or seeming slow, getting into an unusable ...
Read more >Internet Explorer 11 has retired and is officially out of support ...
The retired, out-of-support Internet Explorer 11 desktop application is scheduled to be permanently disabled through a Microsoft Edge update ...
Read more >How to Update Internet Explorer - IE11 - Lifewire
The latest version of Internet Explorer is IE11. Find out what version of Internet Explorer you have if you're not sure if Internet...
Read more >Microsoft Will Retire Internet Explorer 11 on June 15, 2022
Ending support for a browser means that bug patches and security updates will no longer take place. This increases the risk of security...
Read more >Internet Explorer 11 End of Life - Lansweeper.com
The next step in the IE11 end of life will be the retirement of the Internet Explorer 11 desktop application on most supported...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
@stefcameron Awesome work! I have pulled down the latest and everything appears to be working as I would expect 💯. Thanks for the quick turnaround on this issue. 😄
@bparish628 👋 Thanks for reaching out! We definitely transpile the code (and those settings, without any browser targets, should be putting everything back down the ES5) – at least, it’s supposed to be getting transpiled!
However, looking at
./dist/focus-trap.js
in the published package, I can see that it’s in fact not the case, which isn’t the intent.I’ll try to fix this ASAP!