owncloud / core

:cloud: ownCloud web server core (Files, DAV, etc.)
https://owncloud.com
GNU Affero General Public License v3.0
8.39k stars 2.05k forks source link

Get rid of jquery UI #18739

Closed PVince81 closed 6 years ago

PVince81 commented 9 years ago

Libs

Apps

jancborchardt commented 9 years ago

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

georgehrke commented 9 years ago

What's about the date picker when setting an expiration date for a share?

jancborchardt commented 9 years ago

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.

raghunayyar commented 9 years ago

@jancborchardt for calendar we intend to move to popover in rework as soon as we get rid of jquery UI in core.

libasys commented 9 years ago

what's about sortable, drag & drop and so on, all primary usefull functions?

jancborchardt commented 9 years ago

@libasys can you analyze where those functions are used? And those are from jQuery UI, not simply jQuery?

jancborchardt commented 9 years ago

@raghunayyar then basically already move to popover now because we will definitely get rid of jQuery UI for 9.0.

libasys commented 9 years ago

http://jqueryui.com/sortable/ http://jqueryui.com/draggable/ http://jqueryui.com/droppable/ http://jqueryui.com/progressbar/

and so on!

jancborchardt commented 9 years ago

@libasys where are they used?

libasys commented 9 years ago

i think many community apps use this functions! progressbar as example for the import dialog! drag, drop & sortables in contacts app and so on! you can't say get rid of jquery ui, this lib is since a few year in the core integrated! what should community apps devs do in future? i think the first point is get rid of jquery ui dialog but not get rid of jquery ui! the second step get rid of tables for more fluid & responsive design!

jancborchardt commented 9 years ago

@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.

oparoz commented 9 years ago

libasys where are they used?

You can't drag and drop in the files app without it

i think the first point is get rid of jquery ui dialog but not get rid of jquery ui!

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.

georgehrke commented 8 years ago

What's the state on this? Will jQuery UI be removed with 9.1?

PVince81 commented 7 years ago

No, there are still too many components using it unfortunately... Need to sort these out first.

@felixheidecke

PVince81 commented 6 years ago

@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)

PVince81 commented 6 years ago

@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

PVince81 commented 6 years ago

@felixheidecke bad things will already happen when updatingjust from jquery-ui 1.10.0 to 1.10.4, see https://github.com/owncloud/core/pull/30088#issuecomment-410327858

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.

PVince81 commented 6 years ago

@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...

PVince81 commented 6 years ago

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.

PVince81 commented 6 years ago
PVince81 commented 6 years ago

Ticket for the jquery ui upgrade: https://github.com/owncloud/core/issues/32484

felixheidecke commented 6 years ago

@PVince81 can we close this as there is a new upgrade Issue instead of getting rid?