-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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
What is jQuery needed for? #25
Comments
Isn't that just a docs specific dependency? Does this mean that all of the source is implemented under the |
👍 React and JQuery do not play nicely and ideally no React components should rely on JQuery. |
Here's what we're currently using out of the jQuery library: Components:
Mixins & Utils:
Any suggestions on how we could remove the jQuery dependency and still have the above functionality? |
innerHeight() - this.getDOMNode().clientHeight However, while these can be directly substituted, for many of them (like classes, events) React has its own pattern and way of doing things. This will require changing a little logic, but it will make everything run much smoother and allow for easier future development. |
It would be nice to remove the jQuery dependency, since people are currently avoiding to use it because of the overheat in some projects. |
@DUBERT care to elaborate on that, I'd like to know |
I've removed some of the jQuery dependencies. Still working on the others. Here's what's left:
Mixins & Utils:
|
There is one jQuery dependency that I can't get rid of. Currently, I'm using jQuery to namespace click events for the click-awayable mixin. This mixin allows components to do something if a click event happens outside of the component. Anyone have a good way to do this with native JS? Here's the JS file I'm talking about: |
Figured it out. :) |
Move picker sources into lab (#4) Update README.md [DatePicker] Refactor pickers tests to testing-library and mocha (#5) [TimePicker] Migrate tests to testing library (#8) [DateTimePicker] Migrate tests (#9) Fix all pickers linter errors (#10) Fix all circular dependencies (#11) * Fix all circular dependencies * Enable mocha eslint rules for typescript tests [test] The last step to a green CI (#15) Migrate pickers docs (#12) Downgrade to withStyles for pickers sources (#16) Add public api exports for pickers components (#17) Consolidate component namespace and theme augmentation (#18) Describe conformance for pickers sub-components (#19) Autogenerate proptypes for typescript sources (#20) Proper build output (#21) Clear migration artifacts (#23) Eslint rule for lower-case test name convention (#24) DateRangePicker (#25) yarn deduplicate Remove GridListTile [DateTimePicker] Fix migration unit tests Fix types Fix typescript types migration issues Fix yarn lerna build (#33) Fix karma tests use window.Touch for CI karma tests Remove more outdated diff noise (#34) Replace not valid formats with valid ISO strings Try to fix CI touch tests Skip tests if Touch events are not supported Fix merge conflicts Actually type-check Fix safari tests Remove lowercase test name rule The casing is up to the test author. We're not the grammar police in tests. Fix lint Format Remove overzealous eslint-disable* Debug failing tests Better debugging Timezones are fun was isoString th efix? Let's find out what's failing and then skip it Branch for safari Skip DateRangePicker in browsers review Matt's review Co-authored-by: Matt <[email protected]> format docs:i18n
I'd like to use parts of this, but without the jQuery dependency.
I've search for jQuery and found some files that say they require jQuery but never actually use it (like
table-header.jsx
).Is there a list of features jQuery is used for? Or is there a plan to get rid of the dependency?
The text was updated successfully, but these errors were encountered: