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 working on IE11 #76

Closed
SmileyMan opened this issue Aug 6, 2019 · 9 comments
Closed

Not working on IE11 #76

SmileyMan opened this issue Aug 6, 2019 · 9 comments
Assignees
Labels
Milestone

Comments

@SmileyMan
Copy link

Plugin does not work in IE11

Just open Demo in IE11. Maps do not render and logs JavaScript errors

*Sry company I work for still uses this old browser.

SCRIPT1006: Expected ')'
File: jk64reportmap_r1.min.js, Line: 1, Column: 7448

jQuery.Deferred exception: Object doesn't support property or method 'reportmap' TypeError: Object doesn't support property or method 'reportmap'

SCRIPT438: Object doesn't support property or method 'reportmap'
File: desktop_all.min.js, Line: 2, Column: 31040

@jeffreykemp
Copy link
Owner

Thanks for reporting this. I've reproduced this in IE11. I'll work on a fix.

@jeffreykemp
Copy link
Owner

Turns out IE 11 does not support default parameters for javascript functions.

This will be resolved by v1.0.2.

@SmileyMan
Copy link
Author

Thank you.

@jeffreykemp
Copy link
Owner

SmileyMan, please download and install the plugin from this branch:

https://github.com/jeffreykemp/jk64-plugin-reportmap/tree/Patch-1-0-2

and confirm whether it works for you now.

Note: if you need the pre-18.2 version, I haven't updated the backport yet. If so, let me know.

Thanks

@jeffreykemp jeffreykemp added this to the 1.0.2 milestone Aug 6, 2019
@SmileyMan
Copy link
Author

SmileyMan commented Aug 19, 2019

Hi. Sorry been working away.

I have to use 5.03 which is the version we have on our platform at the moment.

@jeffreykemp
Copy link
Owner

No problem - I've backported the fix to 5.0.3, please download the updated plugin from here:

https://github.com/jeffreykemp/jk64-plugin-reportmap/tree/Patch-1-0-2/backport

@SmileyMan
Copy link
Author

SmileyMan commented Aug 27, 2019

Can confirm this now works with IE11 - APEX 5.03

1.0.2

@jeffreykemp
Copy link
Owner

Merged into v1.1

@jeffreykemp
Copy link
Owner

Fixed in Release 1.1

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

2 participants