-
Notifications
You must be signed in to change notification settings - Fork 75
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
Drop jquery dependency #862
Comments
@zachmullen this had come up in the past. we would still need something to replace it. what would be your suggestions? |
For what purpose do you need to replace it? I would suggest using native APIs for element selection & manipulation. If you are talking about for XHR wrapping APIs, axios is a great one. For general why-isn't-this-in-the-standard-library sorts of things, underscore/lodash is nice. |
To clarify, the intent of this is less about replacing jquery with things that meet the same needs, and more about removing jquery from any user-facing aspect of geojs. That is, downstreams of geojs should never have to know whether or not geojs depends on jquery. For example, this part of the docs should go away: And any references to jquery or |
Obligatory reference: http://www.vanilla-js.com/. |
I love that it actually downloads a 0 byte file for you. |
It would be easy to drop jquery from geojs's source. We use jQuery in four ways:
We do use jQuery in some of the examples and tutorials for ajax and selectors. We could continue using it there, or do without as we see fit (but removing it from examples could be done as a separate issue). |
If any of your actual API surface involves jquery |
For modernization
The text was updated successfully, but these errors were encountered: