-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Get rid of jquery UI #18739
Comments
cc @georgehrke @raghunayyar @libasys for Calendar – if jQuery UI is still used there, get rid of it. @blizzz use the tabs as in the sidebar. No accordion of possible. If needed, do it similarly as in the Proxy app or in the installation with simple JS expanding cc @LukasReschke |
What's about the date picker when setting an expiration date for a share? |
Can we isolate that or get it from a library instead? If that’s the only part we use of jQuery UI then it doesn’t warrant pulling in the whole thing. |
@jancborchardt for calendar we intend to move to popover in rework as soon as we get rid of jquery UI in core. |
what's about sortable, drag & drop and so on, all primary usefull functions? |
@libasys can you analyze where those functions are used? And those are from jQuery UI, not simply jQuery? |
@raghunayyar then basically already move to popover now because we will definitely get rid of jQuery UI for 9.0. |
@libasys where are they used? |
i think many community apps use this functions! progressbar as example for the import dialog! |
@libasys yeah, what I was saying is that we should try to isolate which parts of jQuery UI we actually use. And then only ship those. |
You can't drag and drop in the files app without it
It doesn't really matter as it's a library which can be included in the apps themeselves. It's more of a problem when core JS stuff is thrown away. |
What's the state on this? Will jQuery UI be removed with 9.1? |
No, there are still too many components using it unfortunately... Need to sort these out first. |
@felixheidecke please investigate whether we can/should update to the latest jquery.ui version and how much time it would need to migrate (find out with migration guide and function usage) |
@felixheidecke can you prepare and run your script for the two jquery.ui migrations ? we need to evaluate how much work this is compared to getting rid of jquery.ui altogether |
@felixheidecke bad things will already happen when updatingjust from jquery-ui 1.10.0 to 1.10.4, see #30088 (comment) The worse part is that OC has a file jquery-ui-fixes.css that overrides styles which will likely change in future versions. So when updating we'd need to adjust these hacks. |
@felixheidecke let's compare how much time we'd need to update jquery.ui including the existing OC hacks (jquery-ui-fixes.css) with the time it would take to replace all components with non-jquery-ui ones... |
Updated plan: we carry on with updating jquery.ui and will fix jquery.ui.fixes. Considering that Phoenix is the future, it's not worth investing time trying to replace the jquery.ui plugins with alternatives. |
|
Ticket for the jquery ui upgrade: #32484 |
@PVince81 can we close this as there is a new upgrade Issue instead of getting rid? |
Libs
Apps
The text was updated successfully, but these errors were encountered: