Opened 23 months ago

Closed 20 months ago

Last modified 19 months ago

#17952 closed task (blessed) (fixed)

Update to latest jQuery and UI

Reported by: hebbet Owned by: azaozz
Priority: normal Milestone: 3.3
Component: External Libraries Version: 3.2
Severity: normal Keywords: needs-patch
Cc: ramoonus@…, jpenney@…

Description

Attachments (1)

17952.patch (179.2 KB) - added by ocean90 23 months ago.

Download all attachments as: .zip

Change History (29)

  • Cc ramoonus@… added

maybe for 3.2.1

  • Keywords 3.3-early added
  • Milestone changed from Awaiting Review to Future Release

We'll drop this into trunk right after we branch. If things don't break then 3.2.1 is possible.

  • Keywords has-patch added; needs-patch removed
  • Milestone changed from Future Release to 3.3

comment:6 follow-up: ↓ 9   azaozz22 months ago

  • Keywords 3.3-early has-patch removed
  • Owner set to azaozz
  • Status changed from new to reviewing
  • Summary changed from Update to jQuery 1.6.2 to Update to latest jQuery and UI

The update to jQuery 1.6.1 brought quite a few plugin/theme issues despite that we did everything possible to warn plugin and theme authors. Perhaps we should update both jQuery and UI in August and stick with them for 3.3.

related: #17474

comment:8 follow-up: ↓ 10   azaozz22 months ago

Closed #17474 as duplicate.

Replying to jczorkmid:

For compatibility's sake this is a good idea. Was hoping we would move away from UI in core as we mostly use it for drag+drop support and there are lighter jQuery plugins for this.

However it seems many WordPress plugins and themes will continue to use UI, so adding the whole UI to core will make it easier for them. The only "against" would be the increase of the total download size of WP.

On the other hand we probably could un-bundle Prototype.js as very few plugins seem to use it. Perhaps we can leave it in script-loader but load it from Google.

Last edited 22 months ago by azaozz (previous) (diff)

comment:9 in reply to: ↑ 6   ramoonus22 months ago

Replying to azaozz:

The update to jQuery 1.6.1 brought quite a few plugin/theme issues despite that we did everything possible to warn plugin and theme authors. Perhaps we should update both jQuery and UI in August and stick with them for 3.3.

true but was 1.6.1 or 1.6 to blaim?
i think 1.6

comment:10 in reply to: ↑ 8 ; follow-up: ↓ 11   jczorkmid22 months ago

  • Cc jpenney@… added

Replying to azaozz:

Closed #17474 as duplicate.

Should I attach the jQuery UI patch to this ticket so it doesn't get lost in the shuffle?

comment:11 in reply to: ↑ 10   azaozz22 months ago

Replying to jczorkmid:

No need, whoever commits this can grab the latest version from the UI website if there's one or use the patch from #17474.

comment:12 follow-up: ↓ 13   jczorkmid22 months ago

Ok, thanks. Hopefully they won't just grab the latest. The patch also added the remainder of the UI libraries to wp-includes/script-loader.php. :)

Version 0, edited 22 months ago by jczorkmid (next)

comment:13 in reply to: ↑ 12   ramoonus22 months ago

errrrrrrr

Last edited 22 months ago by ramoonus (previous) (diff)
  • Keywords needs-patch added

jQuery 1.6.3 ...

comment:17 follow-up: ↓ 18   marcelometal21 months ago

I updated WP for use jQueryUI >= 1.8.13 and an error occurred when using a new widget in the sidebar.

In wp-admin/js/widgets.js, line 113:

var add = ui.item.find('input.add_new').val(),
    n = ui.item.find('input.multi_number').val(),
    id = ui.item.attr('id'),
    sb = $(this).attr('id');

'id' is undefined, because it was removed from ui.item

http://bugs.jqueryui.com/ticket/4564

https://github.com/jquery/jquery-ui/commit/98fcb477d6a7091b21e9b7bc0b8c28f798954d47#ui/jquery.ui.draggable.js

comment:18 in reply to: ↑ 17   azaozz20 months ago

Replying to marcelometal:

Yes, this is a regression. Still wondering what's better: patch that and update to 1.8.16 or keep 1.8.12. Opened a ticket on UI's trac: http://bugs.jqueryui.com/ticket/7734

comment:19 follow-up: ↓ 21   nacin20 months ago

Would rather patch and update to 1.8.16, but still track this upstream. Thanks for submitting.

In [18757]:

Update jQuery to 1.6.4, update jQuery UI to 1.8.16, add the rest of UI, add workaround for a regression in UI Draggable, see #17952

comment:21 in reply to: ↑ 19   azaozz20 months ago

Replying to nacin:

Yes, added a workaround for that, preferable to changing it. Hopefully it will not break anything else as combining Draggable with a Sortable is not common.

Glad to see the full jQuery UI library rolled in.

comment:23 follow-up: ↓ 24   azaozz20 months ago

In [18774]:

Fix the workaround for UI Draggable to include single instance widgets (do they still exist?), see #17952

comment:24 in reply to: ↑ 23   nacin20 months ago

Replying to azaozz:
single instance widgets (do they still exist?)

They're all over the place, for sure.

Akismet even still uses them.

In [18795]:

Correct the src for jquery-ui-datepicker. props danielbachhuber, fixes #18793. see #17952.

  • Type changed from enhancement to task (blessed)
  • Resolution set to fixed
  • Status changed from reviewing to closed

Resolving as fixed. We can open a new ticket if and when the jui regression is addressed upstream.

Nothing.

Last edited 19 months ago by ocean90 (previous) (diff)
Note: See TracTickets for help on using tickets.