Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#16426 closed enhancement (fixed)

jQuery 1.5.2

Reported by: duck_ Owned by:
Priority: normal Milestone: 3.2
Component: External Libraries Version: 3.1
Severity: normal Keywords: has-patch needs-testing
Cc: chris@…

Attachments (5)

16426.diff (159.4 KB) - added by c3mdigital 2 years ago.
jquery.diff (160.3 KB) - added by niallkennedy 2 years ago.
jQuery 1.5.1 diff against 1.4.4
t16426-jquery-144-151.diff (161.0 KB) - added by demetris 2 years ago.
Updates jQuery from 1.4.4 to 1.5.1
t16426-jquery-144-152.diff (161.6 KB) - added by JDTrower 2 years ago.
Updates jQuery from 1.4.4 to 1.5.2
t16426-jquery-144-152.2.diff (161.6 KB) - added by demetris 2 years ago.
Updates jQuery from 1.4.4 to 1.5.2

Download all attachments as: .zip

Change History (35)

  • Keywords has-patch added
  • Type changed from defect (bug) to enhancement
  • Version set to 3.1
  • Cc chris@… added

Patch added

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

We'll get this in 3.2, early on. Too late for 3.1.

+1 on this one, glad it's getting into 3.2

jQuery 1.5.1 (bugfix release) is planned for next thursday

  • Summary changed from jQuery 1.5 to jQuery 1.5.1

jQuery 1.5.1 has been released yesterday.

Related: #16508

jQuery 1.5.1 diff against 1.4.4

comment:9   ryan2 years ago

  • Keywords 3.2-early removed
  • Milestone changed from Future Release to 3.2
  • Keywords close added
  • Keywords close removed

oops

Updates jQuery from 1.4.4 to 1.5.1

  • Keywords needs-patch added; has-patch removed
  • Summary changed from jQuery 1.5.1 to jQuery 1.5.2

Updates jQuery from 1.4.4 to 1.5.2

  • Keywords has-patch added; needs-patch removed

Updates jQuery from 1.4.4 to 1.5.2

JDTrower, you forgot the noConflict() bit we add at the end. :-)

You're right. I did. I was running a comparison between your diff and mine to figure out what was different. noConflict() slipped my mind to make sure I added it back.

The patch looks good. However, did anybody test if it breaks something? Currently tracking several issues that came from updating jQuery and jQUI in 3.1, one quite annoying that can make the Edit Page screen unusable in IE7/8.

Perhaps some (light) testing is in order before even thinking of updating. Unfortunately there's not much automated tests that we can do on UI sortables, etc. so it seems it has to be done by hand. A good testing install would have 300-400 Posts, 300-400 Pages, 200-300 Tags and 200-300 Cats. Main things to test:

  • All draggable, droppable, sortable elements work as expected on all pages.
  • Everything on the Add/Edit Post/Page screens works as expected.
  • Any bugs are reported on the jQuery and jQUI tracs.
Version 0, edited 2 years ago by azaozz (next)
  • Keywords needs-testing added

and opera 11

  • Component changed from External Libraries to Export
  • Owner set to JDTrower
  • Status changed from new to reviewing

I'm still planning on testing the other browsers, but have tested in FF 4.0 and the same problem reported on #16955 still occurs in 3.2-bleeding with jQuery updated to 1.5.2 with this patch.

  • Component changed from Export to External Libraries
  • Owner JDTrower deleted

Not sure how I got set to owner or how the status or component got changed, but trying to fix that now.

I have tested the latest patch in IE 8, IE 9, FF 3.6, FF 4.0, Safari 5, Chrome 10, Opera 10, and Opera 11 on Windows Vista and Windows 7 computers and I didn't notice any bugs. Everything worked as expected. This was done on a test install with 311 posts, 387 pages, 216 categories, 408 tags, and 401 comments.

Last edited 2 years ago by JDTrower (previous) (diff)

Should have mentioned that IE 8 and 9, I did encounter problems that are listed in detail on #17070. But other than that ticket and #16955 (which the fix has been applied upstream and just waiting on the release of jQuery UI 1.8.12) everything else worked fine for me.

comment:22 follow-up: ↓ 23   demetris2 years ago

Thank you for you taking the time to do this, JDTrower.

By the way, nacin suggested the other day on the IRC channel that we should make a checklist for testing jQuery and jQuery UI updates. I think this is a good idea, as it would make testing better and easier: you open the checklist, launch the supported browsers, and you start going through the checklist point by point.

comment:23 in reply to: ↑ 22   JDTrower2 years ago

Replying to demetris:

Thank you for you taking the time to do this, JDTrower.

By the way, nacin suggested the other day on the IRC channel that we should make a checklist for testing jQuery and jQuery UI updates. I think this is a good idea, as it would make testing better and easier: you open the checklist, launch the supported browsers, and you start going through the checklist point by point.

You're welcome demetris.

I agree that that is a good idea. It would make testing not only better and easier, but also more consistent. I assume a checklist like that would be a Codex page or would it be located somewhere else?

1.6 beta 1 has just been released
more info @ http://blog.jquery.com/2011/04/15/jquery-16-beta-1-released/

jQuery 1.6 is currently planned for end of April.
Feature freeze of Wordpress 3.2 is schedueled for April 30, so i think it is too late for getting jQuery 1.6 into Wordpress 3.2

@JDTrower: Yes, a Codex page would be a good start.

@ramoonus: jQuery 1.6 doesn't sound like a good idea for WP 3.2.

Last edited 2 years ago by scribu (previous) (diff)

i know its slightly offtopic
but i have developed a very simple plugin ( http://wordpress.org/extend/plugins/jquery-updater/ )
which replaces the default WP jQuery with the latest stable version
but only for the front-end
ideal for testing your websites content with a "future" WP jQuery version

  • Resolution set to fixed
  • Status changed from reviewing to closed

(In [17725]) jQuery 1.5.2, props JDTrower demetris, fixes #16426

In [17777]:

Add quotes to [value="some"] selectors, see #16426

For jQuery 1.6: #17311

Note: See TracTickets for help on using tickets.