-
Notifications
You must be signed in to change notification settings - Fork 78
Open
Description
The assumption that a project even includes/uses jQuery is rather unsafe and unusual these days.
But the core of this project is still valuable in the modern post-jQuery Rails world, so it would be nice to keep the good parts and just remove the dependency on jQuery.
Looks like it wouldn't be too hard to rewrite app/assets/javascripts/xray.js without it.
While we're at it, consider:
- migrating from CoffeeScript to a regular modern ES6+ module, and
- making it so wan include the module into our Webpack JS pack (remove the dependency on/assumption of Sprockets too)...
See also:
- Removing the auto-adding of xray.js after jquery.js (Remove auto-adding after jquery.js script — Sprockets 4 (nor webpack-based javascript_pack_tag 'application') doesn't add a separate script tag for jquery #110)
tonydehnke, QuintinAdam and 007lva
Metadata
Metadata
Assignees
Labels
No labels