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

wrong polyfill with IE11 ? #9516

Closed
cvlmtg opened this issue Feb 27, 2018 · 2 comments · Fixed by #9520
Closed

wrong polyfill with IE11 ? #9516

cvlmtg opened this issue Feb 27, 2018 · 2 comments · Fixed by #9520
Labels

Comments

@cvlmtg
Copy link

cvlmtg commented Feb 27, 2018

schermata 2018-02-27 alle 15 11 26

Configuration:

  • Web browser and its version: IE11
  • Operating system and its version: Windows 7
  • PDF.js version: not sure, but probably >= 2.0.0
  • Is a browser extension: no

Steps to reproduce the problem:

  1. open https://mozilla.github.io/pdf.js/web/viewer.html
  2. try to use Array.prototype.includes()

What is the expected behavior? (add screenshot)

[1,2].includes(1) should return true, not false

@yurydelendik
Copy link
Contributor

yurydelendik commented Feb 27, 2018

fwiw, the polyfill comes from core-js library

@cvlmtg
Copy link
Author

cvlmtg commented Feb 27, 2018

@yurydelendik unless I completely misunderstood something, according to the core-js author the fault is not in his polyfill:

babel/babel#7419

(please look the last comment on that issue)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants