-
Notifications
You must be signed in to change notification settings - Fork 12
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
Security: Upgrade to jQuery UI 1.13.0 #46
Comments
I tried following the update steps described in: https://github.com/collective/collective.js.jqueryui/blob/master/docs/UPGRADE.txt However, the This hampers the choice plugins functionality, described in: https://github.com/collective/collective.js.jqueryui#include-plugins-and-optimizations One option would be to clone the jqueryui repository and build it to generate the files. But that makes things more complicated. So I ask if we can remove this functionality, or if it is really important. |
Being able to slim the javascript down is a useful option. But upstream has made this impossible/difficult, then we may need to remove this. Would need be a a major breaking release I would say. |
I agree. If removed it simply should be a new major release with a hint in the README. |
Something like this? |
@szakitibi would you like to contribute to the https://github.com/collective/Collective/issues and ask to add to the collective organization. So you can create a branch here in this repository. |
collective/Collective#1785 (comment) :D Well to avoid complications I simply created the pull request see #56. |
The text was updated successfully, but these errors were encountered: