Accessing the 'type' property on VML elements fails on IE. Fixes #7071.

0d8b247c · John Firebaugh · John Resig · 3418f323 · 0d8b247c · 0d8b247c
Commit 0d8b247c authored Apr 12, 2011 by John Firebaugh Committed by John Resig Apr 12, 2011
Hide whitespace changes
Inline Side-by-side

Showing with 10 additions and 2 deletions

event.js src/event.js +2 -2

index.html test/index.html +5 -0

event.js test/unit/event.js +3 -0

No files found.
--- a/src/event.js
+++ b/src/event.js
@@ -800,7 +800,7 @@ if ( !jQuery.support.changeBubbles ) {
 			beforedeactivate: testChange,

 			click: function( e ) {
-				var elem = e.target, type = elem.type;
+				var elem = e.target, type = jQuery.nodeName( elem, "input" ) ? elem.type : "";

 				if ( type === "radio" || type === "checkbox" || elem.nodeName.toLowerCase() === "select" ) {
 					testChange.call( this, e );
@@ -810,7 +810,7 @@ if ( !jQuery.support.changeBubbles ) {
 			// Change has to be called before submit
 			// Keydown will be called before keypress, which is used in submit-event delegation
 			keydown: function( e ) {
-				var elem = e.target, type = elem.type;
+				var elem = e.target, type = jQuery.nodeName( elem, "input" ) ? elem.type : "";

 				if ( (e.keyCode === 13 && elem.nodeName.toLowerCase() !== "textarea") ||
 					(e.keyCode === 32 && (type === "checkbox" || type === "radio")) ||

--- a/test/index.html
+++ b/test/index.html
@@ -45,6 +45,10 @@
 	<script src="unit/effects.js"></script>
 	<script src="unit/offset.js"></script>
 	<script src="unit/dimensions.js"></script>
+
+	<!-- For testing http://bugs.jquery.com/ticket/7071 -->
+	<xml:namespace ns="urn:schemas-microsoft-com:vml" prefix="v" />
+	<style>v\:oval { behavior:url(#default#VML); display:inline-block; }</style>
 </head>

 <body id="body">
@@ -147,6 +151,7 @@
 			<span id="test.foo[5]bar" class="test.foo[5]bar"></span>

 			<foo_bar id="foobar">test element</foo_bar>
+			<v:oval id="oval" style="width:100pt;height:75pt;" fillcolor="red"> </v:oval>
 		</form>
 		<b id="floatTest">Float test.</b>
 		<iframe id="iframe" name="iframe"></iframe>

--- a/test/unit/event.js
+++ b/test/unit/event.js
@@ -776,6 +776,9 @@ test("trigger() shortcuts", function() {

 	// manually clean up detached elements
 	elem.remove();
+
+	// test that special handlers do not blow up with VML elements (#7071)
+	jQuery("#oval").click().keydown();
 });

 test("trigger() bubbling", function() {