-
Notifications
You must be signed in to change notification settings - Fork 804
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 #16409
Comments
+1 |
Noting that any effort in this direction will most likely help avoiding any conflicts that may come up when Core updates the version of jQuery bundled with WordPress: |
I'll have a go at Contact Forms
|
I'm starting to work on: Tiled Gallery
|
@dero @davidlonjon - if you're working on something please update the description to put your name on it, and if possible a link to your PR, so we don't overlap. |
@davidlonjon Just make sure you aren't removing jQuery as a dep from anything shown in wp-admin. I am not really interested in wp-admin right now - only front-end code. |
Ok thanks for the feedback. For info I have not done much work on this as I refocused my attention on our other project. |
Moved |
Noting here that we should use the wp-dom-ready package instead of https://developer.wordpress.org/block-editor/packages/packages-dom-ready/ |
I had a go at |
@gravityrail: Adding a note here that once the plan we've been discussing in #16592 (comment) has been finalised, we should probably document it and add a link to it on this issue. |
Noting that the Same in It may be worth prioritizing these files (and others that trigger such warnings) accordingly. |
@sgomes I know you had worked on removing jQuery dependencies at some point. Is that something that's still on you radar today? |
@jeherve I haven't looked at this in a while, but I seem to recall Perhaps a new audit of |
FYI for Forms: |
What follows is a list of files that we have detected jQuery-like syntax in, listed in approximate priority order.
Prioritisation was based on the following(rough) criteria:
In eliminating these uses of jQuery, you may find this web site useful: http://youmightnotneedjquery.com/
Files
The following list of files was generated using eslint. Don't forget, once you modify a file, to remove
jquery
as a dependency wherever that file was enqueued.Front-end
Carousel
Tiled Gallery
Related Posts
Sharing Buttons
Gutenberg
Contact Forms
Contact Info Widget
EU Cookie Widget
Likes
Masterbar
Post by Email
Misc Shortcodes (rarely used)
Responsive Videos
Site Logo Control
Widget Visibility
modules/widget-visibility/widget-conditions/widget-conditions.js
modules/widgets/gallery/js/admin.js
modules/widgets/milestone/admin.js
modules/widgets/social-icons/social-icons-admin.js
modules/widgets/twitter-timeline-admin.js
Comics
VideoPress
Customizer
CSS Editor
Content Options
Customizer Utils
Simple Payments
Infinite Scroll
WP-Admin
It might not be worth bothering removing jQuery from wp-admin, as on these pages performance is less of an issue, and they prrrrobably enqueue jQuery no matter what you do.
Comment Likes
Just in Time Messages
Notices
Jetpack Settings
Connection
Deactivate warning
"Identity Crisis"
Gallery settings
Plugin Search (unused)
MISC
I don't yet know the scope of these:
The text was updated successfully, but these errors were encountered: