-
Notifications
You must be signed in to change notification settings - Fork 8
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
jQuery as an optional dependency #40
Comments
Been discussing this recently with @matteofigus and I'm totally onboard with this. |
Sounds great, to me. I will see if I can at least get some idea of the internals by looking at adding a unit test or something smaller. |
Totally!
Thoughts? |
|
On API compatibility: Agreed I think we would need to maintain decent unit / integration tests for that. I think that would be a requirement or there would have to be just another browser client entirely. I think we could deal with the Polyfilling: I really like the approach the stencil team is using for automatically feature detecting and loading only on demand. It might be a lot better to do feature detection over the FT approach of user agent sniffing. Further discussions might be to invest some in modernizing build / tests as well if we are going to invest in the core.
|
How is this going? Could lend a hand |
Why?
The overhead of loading all of jquery is not necessary for the oc-client-browser. Many of the same things can be done using the built in platform and polyfilling for legacy support. I would like to think about how we start to modernize the oc-client-browser to be jquery free and as small and light as possible.
The text was updated successfully, but these errors were encountered: