From 746074f0f74b0c0916fb2db2b6370c324822f31a Mon Sep 17 00:00:00 2001 From: Dave Methvin Date: Mon, 24 Oct 2011 18:05:53 -0400 Subject: Remove attribute match from quickIs As @timmywil points out, attributes and properties are confused by IE6/7. This commit also reworks the unit test case to do a better job of checking className matches. --- test/unit/event.js | 25 +++++++------------------ 1 file changed, 7 insertions(+), 18 deletions(-) (limited to 'test') diff --git a/test/unit/event.js b/test/unit/event.js index de9c225c1..35664b022 100644 --- a/test/unit/event.js +++ b/test/unit/event.js @@ -2409,11 +2409,11 @@ test(".on and .off", function() { }); test("delegated events quickIs", function() { - expect(17); + expect(14); var markup = jQuery( '
'+ '

'+ - 'deadbeatclub'+ + 'deadbeatclub'+ '

'+ ''+ 'workedorborked?'+ @@ -2437,29 +2437,18 @@ test("delegated events quickIs", function() { .appendTo( "body" ) .on( "blink", "em", func ) .on( "blink", ".D", func ) + .on( "blink", ".devo-like", func ) + .on( "blink", ".devo", func ) .on( "blink", ".d", func ) .on( "blink", "p.d", func ) - .on( "blink", "[devo=cool]", func ) - .on( "blink", "[devo='NO']", func ) .on( "blink", "#famous", func ); - check( "[devo=cool]", "b|[devo=cool] p|.D" ); - check( "[devo='']", "" ); + check( ".devo-like", "b|.devo-like p|.D" ); + check( ".devo", "" ); check( "p", "p|.D" ); - check( "b", "b|[devo=cool] p|.D" ); + check( "b", "b|.devo-like p|.D" ); check( "em", "em|em q|#famous em|em q|#famous" ); - markup.find( "b" ).attr( "devo", "NO" ); - check( "b", "b|[devo='NO'] p|.D" ); - - markup - .on( "blink", ".tricky", function() { - ok( false, "triggered on wrong class name match" ); - }) - .find( "p" ) - .attr( "class", "tricky-match" ) - .trigger( "blink" ); - markup.remove(); }); -- cgit v1.2.3