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

Remove jQuery dependency #46

Closed
2 tasks
mejackreed opened this issue Sep 27, 2016 · 4 comments
Closed
2 tasks

Remove jQuery dependency #46

mejackreed opened this issue Sep 27, 2016 · 4 comments

Comments

@mejackreed
Copy link
Owner

@epoz
Copy link

epoz commented Dec 12, 2017

I made another attempt at: epoz@88fa7ed using the fetch() API.

@mejackreed comments on Slack:
"Without any arrow function support for Internet Explorer, I think this may be a non-starter until that can come. I would be more inclined to adopting something like the approach in #45 for maximum compatibility."

My goal with this is to use leaflet-iiif in a React app, where I do not want to also load in jQuery just for this one component.

@abrin
Copy link
Contributor

abrin commented Oct 20, 2019

would you be more open to pulling out jQuery now that two years have passed? I wonder if the active browser support has rolled along enough that the older browser versions aren't as much of an issue (IE 11 at ~2% according to https://caniuse.com/#feat=arrow-functions)

@mejackreed
Copy link
Owner Author

@abrin yes. See #77 for the implementation. I was hoping to get some feedback on that.

@mejackreed
Copy link
Owner Author

Resolved in #77

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

No branches or pull requests

3 participants