Update to jQuery 1.6.1

[scribu]

jQuery 1.6 was ​released today.

This update will require more testing than usual, since some breaking changes were made to the commonly used .attr() function.

[scribu]

Re testing: ​http://core.trac.wordpress.org/ticket/16426#comment:16

[aaroncampbell]

It looks like there are several places that we should be using .prop() instead of .attr(). I'm a little unclear so someone should verify, but it seems that most places where we are setting an attribute we should actually be setting the property. Additionally several places where we check an attribute (especially "checked" and "disabled") we should be actually be checking the property. However, I think everywhere we're checking id, name, etc we should continue using attr().

• ​http://api.jquery.com/.prop/
• ​http://api.jquery.com/attr/
• ​http://api.jquery.com/category/version/1.6/

[nacin]

Suggesting as a candidate for 3.2. Note it also fixes #16643.

[Viper007Bond]

Replying to nacin:

Suggesting as a candidate for 3.2. Note it also fixes #16643.

We should get it in ASAP then.

[dd32]

Suggesting as a candidate for 3.2.

We've already got breakage for back-compat with 1.5.2 (" in selectors), Leaving this out of 3.2 will mean when 3.3 is released, some authors will have to update their JavaScript yet again to support the 1.6 changes. We still have a fair amount of time left for bugfixing, so it's not out of the question to include 1.6..

[scribu]

I would suggest waiting at least for 1.6.1 (which should be released this week), since it will likely contain a fix for the .attr('checked') and .attr('selected') debacle.

Can't find where I read John Resig mentioning that.

[nacin]

​http://news.ycombinator.com/item?id=2510202

[scribu]

When updating from 1.5.2 to 1.6.1, you should not have to change any code.

​http://blog.jquery.com/2011/05/10/jquery-1-6-1-rc-1-released/

[azaozz]

Agree that the sensible thing would be to add 1.6.1 to 3.2, hopefully it will be released on time. We will have to update jQuery UI after that too.

[aaroncampbell]

I was in the jquery-dev channel this morning and their plan was to release 1.6.1 tomorrow. I'm not sure that's still on track because they found a regression that affects Drupal. I looked through and it doesn't look like it would affect us, but just in case here it is:
I happens most often when using ajax to update parts of the document, but the root cause is really when injecting a script snippet into the page. It only happens if there is a CDATA section that is commented like this:

<!--//--><![CDATA[//><!--
...
//--><!]]>

The issue is the html comment around a js comment immediately before the CDATA.

John Resig said that doing it the proper way still works. This was what he said was 'proper':

// <![CDATA[
...
// ]]>

I asked specifically about using block comments (which we do in several places) and they said that would still work as well:

/* <![CDATA[ */
...
/* ]]> */

Most of their comments aren't on the ticket yet, but here it is: ​http://bugs.jquery.com/ticket/9221

[jane]

Punting for now b/c past feature freeze, may revisit if 1.6.1 comes out soon enough in beta.

[aaroncampbell]

Just an update: The bug I mentioned above has been fixed (​http://bugs.jquery.com/ticket/9221). Hopefully that means 1.6.1 will be here soon.

[Viper007Bond]

​http://blog.jquery.com/2011/05/12/jquery-1-6-1-released/

[djibs]

jQuery 1.6.1 was released today.

​http://blog.jquery.com/2011/05/12/jquery-1-6-1-released/

[scribu]

Yes, we've heard.

[JDTrower]

Updates jQuery from 1.4.4 to 1.6.1

[JDTrower]

Getting ready to begin testing that patch to ensure things work based on the list found at ​http://core.trac.wordpress.org/ticket/16426#comment:16 I'll also work at getting a Codex page up that details that list so we can ensure that testing follows a good checklist.

[azaozz]

Fixed few small bugs that showed with 1.6.1 in core and putting it in. @JDTrower perhaps better to make sure latest UI works well with 1.6.1. Seems we will have that combination in 3.2 if no blockers there.

[azaozz]

In [17907]:

jQuery 1.6.1 and fix for some small bugs that popped up while testing it, props JDTrower, fixes #17311

[JDTrower]

@azaozz, I tested both jQuery 1.6.1 and jQuery UI 1.8.12 with the latest Trunk using my test install 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. Everything dragged and dropped like it should, and there were no flickering bugs.

[azaozz]

Replying to JDTrower:

Thanks a lot for checking them and confirming all works as expected. Very much appreciate it.

[JDTrower]

Replying to azaozz:

Not a problem, my pleasure.

[azaozz]

In [18027]:

Replace .attr() with .prop() as per jQuery 1.6.1. Also fixes couple places where the JS wasn't working because attr('some', ) does not remove the HTML property now, see #17311