- 🇺🇸United States solotandem
This request seems quite problematic given the only JS file in the project relies on core
drupalSetSummaryfunction which, even in branch 10.3.x, relies on jQuery. - 🇺🇸United States kmonty San Francisco, CA
I think it makes sense to remove the dependency. Core can require it, but the module itself doesn't need to. Eventually the goal is to remove the jQuery dependency from core (as I understand it), so it seems reasonable to get ahead of that.
- 🇩🇪Germany Anybody Porta Westfalica
What's the state in 2.x? That's where development should happen. Is jQuery already removed there? Then this might be closed outdated and people could switch to 2.x!
- 🇺🇸United States kmonty San Francisco, CA
Great call, @anybody! I just looked at the 2.x branch and jQuery isn't used / required as a library. I'll leave it up to the maintainers to close this as outdated, but it strikes me as something that could be.
- First commit to issue fork.
- 🇨🇦Canada joelseguin Ontario, Canada
I've tested with a fresh D10 install and the 2.x branch and it is definitely loading the jQuery library (from /core/assets/vendor/jquery/jquery.min.js).
To reproduce:
- Disable CSS and JS aggregation from the Drupal UI
- Add an actual Google Analytics (G-xxxxxxxx) tag ID
- You will notice the jquery.min.js file being loaded in the front-end of the site
I would be happy to test a patch if someone has a chance to work on one.