Skip to content
Permalink
Browse files

Use getAttributeNode for ^on attributes in IE6/7 to avoid anonymous f…

…unction wrapper. Fixes #9298.
  • Loading branch information...
timmywil
timmywil committed May 16, 2011
1 parent 6171e0a commit a5cf257a8a240f96f1ec255599fa1d1190f51ff5
Showing with 6 additions and 3 deletions.
  1. +1 −1 src/attributes.js
  2. +5 −2 test/unit/attributes.js
@@ -7,7 +7,7 @@ var rclass = /[\n\t\r]/g,
rfocusable = /^(?:button|input|object|select|textarea)$/i,
rclickable = /^a(?:rea)?$/i,
rboolean = /^(?:autofocus|autoplay|async|checked|controls|defer|disabled|hidden|loop|multiple|open|readonly|required|scoped|selected)$/i,
rinvalidChar = /\:/,
rinvalidChar = /\:|^on/,
formHook, boolHook;

jQuery.fn.extend({
@@ -40,7 +40,7 @@ test("jQuery.attrFix/jQuery.propFix integrity test", function() {
});

test("attr(String)", function() {
expect(39);
expect(40);

equals( jQuery("#text1").attr("type"), "text", "Check for type attribute" );
equals( jQuery("#radio1").attr("type"), "radio", "Check for type attribute" );
@@ -119,7 +119,10 @@ test("attr(String)", function() {
ok( jQuery().attr("doesntexist") === undefined, "Make sure undefined is returned when no element is there." );

var $form = jQuery("<form class='something'></form>").appendTo("#qunit-fixture");
equal( $form.attr("class"), "something", "Retrieve the class attribute on a form" );
equal( $form.attr("class"), "something", "Retrieve the class attribute on a form." );

var $a = jQuery("<a href='#' onclick='something()'>Click</a>").appendTo("#qunit-fixture");
equal( $a.attr("onclick"), "something()", "Retrieve ^on attribute without anonymous function wrapper." );
});

if ( !isLocal ) {

0 comments on commit a5cf257

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