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

'Map is not defined' #10268

Closed
anggikolo11 opened this issue Nov 19, 2018 · 3 comments
Closed

'Map is not defined' #10268

anggikolo11 opened this issue Nov 19, 2018 · 3 comments
Labels

Comments

@anggikolo11
Copy link

Attach (recommended) or Link to PDF file here:

Configuration:

  • Web browser and its version:
    Opera Mini for Android (V 27.0.2254.118423)
  • Operating system and its version:
    Android 5.0.1
  • PDF.js version:
    Latest build
  • Is a browser extension:
    No

Steps to reproduce the problem:

  1. When I tried to open mozilla.github.io/pdf.js/web/viewer.html it can't show pdf. There was a notification 'An error occured while loading the PDF.'
  2. I tried to click 'More information' button and said 'PDF.js v2.1.70 (build: 6ebdbb2) Message: Map is not defined)

What is the expected behavior? (add screenshot)

What went wrong? (add screenshot)
screenshot_2018-11-19-17-07-24

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

@timvandermeij
Copy link
Contributor

It looks like we need a core-js polyfill for Opera Mini then :(

@Snuffleupagus
Copy link
Collaborator

Opera Mini for Android (V 27.0.2254.118423)

Looking at https://en.wikipedia.org/wiki/Opera_Mini, that version doesn't look very up-to-date.

It looks like we need a core-js polyfill for Opera Mini then :(

Looking at https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Map#Browser_compatibility, basically every remotely modern browser[1] has at least basic support for Map.

Hence it seems, at least to me, that the most appropriate resolution is to WONTFIX this issue.


[1] That even includes IE11.

@timvandermeij
Copy link
Contributor

Oh, the current version is 37 according to Wikipedia, not 27; I misread that. Indeed, then we can close this as won't-fix.

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

No branches or pull requests

3 participants