Make WordPress Core

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#19326 closed task (blessed) (fixed)

jQuery 1.7.1

Reported by: ocean90's profile ocean90 Owned by: azaozz's profile azaozz
Milestone: 3.3 Priority: high
Severity: normal Version: 3.3
Component: External Libraries Keywords: has-patch needs-testing commit
Focuses: Cc:

Description

Attachments (1)

19326.patch (183.7 KB) - added by ocean90 14 years ago.

Download all attachments as: .zip

Change History (14)

@ocean90
14 years ago

#1 @nacin
14 years ago

This should be considered a blocker for beta 4.

#2 @nacin
14 years ago

  • Keywords commit added
  • Priority changed from normal to high

#3 follow-up: @westi
14 years ago

  • Owner set to azaozz
  • Status changed from new to reviewing

Andrew please can you review and work the jQuery magic here.

#4 @jane
14 years ago

The text written by @markjaquith for the post-update screen says "WordPress now includes the entire jQuery UI stack and the latest version of jQuery: 1.7."

Has there been testing with 1.7.1? Dropping it in as we're about to release sounds like a risk unless I'm missing something.

#5 @westi
14 years ago

  • Keywords commit removed

One question:

  • What is the compelling bug fixed in 1.7.1 that means we need this for Beta 4?

If there isn't one it can wait for 3.3.1

#6 in reply to: ↑ 3 ; follow-up: @azaozz
14 years ago

Replying to westi:
Did part of this yesterday, continuing today. Looking at the changelog, don't see any bugfixes that affect us directly.

On the other hand probably not a good idea to upgrade to 1.7.1 in a dot release, we keep these for blockers or regression bugs only.

#7 in reply to: ↑ 6 @westi
14 years ago

  • Keywords punt added

Replying to azaozz:

Replying to westi:
Did part of this yesterday, continuing today. Looking at the changelog, don't see any bugfixes that affect us directly.

On the other hand probably not a good idea to upgrade to 1.7.1 in a dot release, we keep these for blockers or regression bugs only.

In that case we should forget about 1.7.1 for WP 3.3 / 3.3.1 and punt this ticket as there is nothing fixed that affects us.

We could consider 1.7.1 or 1.7.x for 3.3.x if a serious bug was found and fixed that caused a serious issue in WP.

#8 @georgestephanis
14 years ago

Here's the previous ticket for discussion of which jQuery version to bundle from a couple weeks back: #19144

#9 @ocean90
14 years ago

If we punt it we need to change the string on about screen.

But I don't see the point, why we should punt this.

#10 @ryan
14 years ago

Even though core might not benefit from the fixes, plugins might. Just something to consider.

#11 @jane
14 years ago

  • Keywords commit added; punt removed

If it's going in then let's get it in now so we have a little time to find out if there will be any conflicts.

#12 @azaozz
14 years ago

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

In [19423]:

jQuery 1.7.1, fixes #19326

Note: See TracTickets for help on using tickets.