jQuery 1.6 is out, and unusually for jQuery, there's a pretty big API compatibility break with regard to the attr
function. Details in the release notes, but basically, attr
really is just for attributes now, whereas before it sort of conflated attributes and properties. There's a new prop
function for interacting with properties.
Update 2011/05/11: jQuery 1.6.1 changes attr
again to make it more backward-compatible for older code, details in the jQuery 1.6.1 blog post.
It's a seemingly-small change, but I've seen a lot of code that will be affected, especially around the "checked" property. If you've been checking whether a checkbox is checked using code along the lines of
// In an event handler, etc.:
if ($(this).attr('checked')) {
// It's checked
}
// Or if you already have a jQuery object for the checkbox:
if (foo.attr('checked')) {
// It's checked
}
...you'll need to update your code. Personally, I prefer going straight to the DOM element (that's what the jQuery team recommends as well):// In an event handler, etc.:
if (this.checked)) {
// It's checked
}
// Or if you already have a jQuery object for the checkbox:
if (foo[0] && foo[0].checked) {
// It's checked
}
...and the above has the advantage of working regardless of what version of jQuery you're using. But you could use prop
instead if you like:// In an event handler, etc.:
if ($(this).prop('checked')) {
// It's checked
}
// Or if you already have a jQuery object for the checkbox:
if (foo.prop('checked')) {
// It's checked
}
(There's also foo.is(':checked')
, but that's awfully round-about.)The big properties I see people stumbling over are "checked" (hence the above) and "value" (but hopefully you were already using val
anyway).
7 comments:
What would you recommend as a backward compatible method for enabling and disabling an input field?
Thanks
Petr
@PePa: For checking state, probably the same as for `checked` above. For changing state, I use a tiny plug-in I dashed off that sets the `disabled` property on the matched elements that have it; example.
A "fix" is coming already: http://news.ycombinator.com/item?id=2512473
In getter-style invocation of boolean properties, attr() will - as of 1.6.1 - return the attribute name in lowercase if the property is true.
So your breaking example of "if ($el.attr('checked'))" will work once again by virtue of the string "checked" evaluating as truthy in JavaScript. What a mess.
Thanks for your plug-in T.J.
Here's perhaps something shorter, inspired by a code from here:
http://news.ycombinator.com/item?id=2511733
$.fn.disable = function(bool) {
if (bool) {
this.attr('disabled', 'disabled');
} else {
this.removeAttr('disabled');
}
}
@crescentfresh: OMG
@PePa: Nice! But I prefer to go straight to the property rather than further indirection.
It is amazing how often you see code like: $(this).attr('...') to access a DOM property, which is increadibly inefficient. At least the jQuery team gets a tick for finally addressing some fundamental issues with attr.
@RobG: Yeah. I was glad to see the docs for prop actually show an example of testing properties directly (if (elem.checked)). I've pointed a couple of people at this article but always intended to do my own version of it. This may be a good time, as prop offers very little on top of a standard loop and direct access. (Unlike attributes, which are a mess cross-browser, where attr really helps you out.)
Post a Comment