Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#17952 closed task (blessed) (fixed)

Update to latest jQuery and UI

Reported by: hebbet's profile hebbet Owned by: azaozz's profile azaozz
Milestone: 3.3 Priority: normal
Severity: normal Version: 3.2
Component: External Libraries Keywords: needs-patch
Focuses: Cc:

Description

Attachments (1)

17952.patch (179.2 KB) - added by ocean90 13 years ago.

Download all attachments as: .zip

Change History (29)

#1 @ramoonus
13 years ago

  • Cc ramoonus@… added

#2 @ramoonus
13 years ago

maybe for 3.2.1

#3 @nacin
13 years ago

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

@ocean90
13 years ago

#4 @ocean90
13 years ago

  • Keywords has-patch added; needs-patch removed

#5 @ryan
13 years ago

  • Milestone changed from Future Release to 3.3

#6 follow-up: @azaozz
13 years 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.

#7 @hebbet
13 years ago

related: #17474

#8 follow-up: @azaozz
13 years 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 13 years ago by azaozz (previous) (diff)

#9 in reply to: ↑ 6 @ramoonus
13 years 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

#10 in reply to: ↑ 8 ; follow-up: @jczorkmid
13 years 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?

#11 in reply to: ↑ 10 @azaozz
13 years 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.

#12 follow-up: @jczorkmid
13 years 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. :)

Last edited 13 years ago by jczorkmid (previous) (diff)

#13 in reply to: ↑ 12 @ramoonus
13 years ago

errrrrrrr

Last edited 13 years ago by ramoonus (previous) (diff)

#16 @ramoonus
13 years ago

  • Keywords needs-patch added

jQuery 1.6.3 ...

#17 follow-up: @marcelometal
13 years 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

#18 in reply to: ↑ 17 @azaozz
13 years 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

#19 follow-up: @nacin
13 years ago

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

#20 @azaozz
13 years ago

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

#21 in reply to: ↑ 19 @azaozz
13 years 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.

#22 @Viper007Bond
13 years ago

Glad to see the full jQuery UI library rolled in.

#23 follow-up: @azaozz
13 years ago

In [18774]:

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

#24 in reply to: ↑ 23 @nacin
13 years ago

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

They're all over the place, for sure.

Akismet even still uses them.

#25 @nacin
13 years ago

In [18795]:

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

#26 @ryan
13 years ago

  • Type changed from enhancement to task (blessed)

#27 @ryan
13 years ago

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

#28 @Anton Torvald
13 years ago

Nothing.

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