jeffreykemp / jk64-plugin-reportmap

Report Google Map APEX Plugin
https://jeffreykemp.github.io/jk64-plugin-reportmap/
MIT License
42 stars 16 forks source link

Not working on IE11 #76

Closed SmileyMan closed 4 years ago

SmileyMan commented 5 years ago

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 commented 5 years ago

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

jeffreykemp commented 5 years ago

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

This will be resolved by v1.0.2.

SmileyMan commented 5 years ago

Thank you.

jeffreykemp commented 5 years ago

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

SmileyMan commented 5 years ago

Hi. Sorry been working away.

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

jeffreykemp commented 5 years ago

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 commented 5 years ago

Can confirm this now works with IE11 - APEX 5.03

1.0.2

jeffreykemp commented 4 years ago

Merged into v1.1

jeffreykemp commented 4 years ago

Fixed in Release 1.1