Skip to content
Permalink
Browse files

Some minor cleanup of the last commit.

  • Loading branch information
jeresig committed Sep 27, 2010
1 parent 97d468f commit d558e9eb6f44783bb06666617c758408daf53e3d
Showing with 4 additions and 5 deletions.
  1. +1 −1 src/attributes.js
  2. +3 −4 src/support.js
@@ -164,7 +164,7 @@ jQuery.fn.extend({
var option = options[ i ];

// Don't return options that are disabled or in a disabled optgroup
if ( option.selected && (jQuery.support.optDisabled ? !option.disabled : option.getAttribute("disabled") === null ) &&
if ( option.selected && (jQuery.support.optDisabled ? !option.disabled : option.getAttribute("disabled") === null) &&
(!option.parentNode.disabled || !jQuery.nodeName( option.parentNode, "optgroup" )) ) {

// Get the specific value for the option
@@ -13,16 +13,15 @@
div.innerHTML = " <link/><table></table><a href='/a' style='color:red;float:left;opacity:.55;'>a</a><input type='checkbox'/>";

var all = div.getElementsByTagName("*"),
a = div.getElementsByTagName("a")[0];
a = div.getElementsByTagName("a")[0],
select = document.createElement("select"),
opt = select.appendChild( document.createElement("option") );

// Can't get basic test support
if ( !all || !all.length || !a ) {
return;
}

var select = document.createElement("select");
var opt = select.appendChild( document.createElement("option") );

jQuery.support = {
// IE strips leading whitespace when .innerHTML is used
leadingWhitespace: div.firstChild.nodeType === 3,

0 comments on commit d558e9e

Please sign in to comment.
You can’t perform that action at this time.