-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Core: IE11 workaround for currentScript #2104
Core: IE11 workaround for currentScript #2104
Conversation
Thanks for pointing that out 😅 |
Hey - I wanted to use Prism embedded into a web component, but ran into the issue that this IE11 hack is used. Specifically it is that in my environment Any thoughts on accepting a PR that extends the |
@muenzpraeger How can |
It runs within a secure context of that environment. |
Ok, after a dive into the documentation, I found the issue. The "global" My conclusion is that this is a bug with |
This fixes #2102.
The IE11 workaround was inspired by this.