Permalink
Browse files

Event: Move VML test out of event alias test

  • Loading branch information...
dmethvin committed Nov 7, 2015
1 parent 7d44d7f commit 67fa2eab6ef323b1d894e9e7f054c6e8c844d304
Showing with 15 additions and 5 deletions.
  1. +15 −5 test/unit/event.js
View
@@ -2833,11 +2833,6 @@ QUnit[ jQuery.fn.click ? "test" : "skip" ]( "trigger() shortcuts", function( ass
};
jQuery( "#simon1" ).click();
assert.equal( clickCounter, 1, "Check that click, triggers onclick event handler on an a tag also" );
// test that special handlers do not blow up with VML elements (#7071)
jQuery( "<xml:namespace ns='urn:schemas-microsoft-com:vml' prefix='v' />" ).appendTo( "head" );
jQuery( "<v:oval id='oval' style='width:100pt;height:75pt;' fillcolor='red'> </v:oval>" ).appendTo( "#form" );
jQuery( "#oval" ).click().keydown();
} );
QUnit[ jQuery.fn.click ? "test" : "skip" ]( "Event aliases", function( assert ) {
@@ -2858,6 +2853,21 @@ QUnit[ jQuery.fn.click ? "test" : "skip" ]( "Event aliases", function( assert )
} );
} );
// Support: IE9 (remove when IE9 is no longer supported)

This comment has been minimized.

Show comment
Hide comment
@mgol

mgol Nov 9, 2015

Member

I don't think this remark in the brackets is necessary; we'd have to put it near almost every support comment. Writing:

// Support: IE 9 only

would be enough.

@mgol

mgol Nov 9, 2015

Member

I don't think this remark in the brackets is necessary; we'd have to put it near almost every support comment. Writing:

// Support: IE 9 only

would be enough.

// https://msdn.microsoft.com/en-us/library/hh801223(v=vs.85).aspx
QUnit.test( "VML with special event handlers (trac-7071)", function( assert ) {
assert.expect( 1 );
var ns = jQuery( "<xml:namespace ns='urn:schemas-microsoft-com:vml' prefix='v' />" ).appendTo( "head" );
jQuery( "<v:oval id='oval' style='width:100pt;height:75pt;' fillcolor='red'> </v:oval>" ).appendTo( "#form" );
jQuery( "#form" ).on( "keydown", function() {
assert.ok( true, "no error was thrown" );
} );
jQuery( "#oval" ).trigger( "click" ).trigger( "keydown" );
ns.remove();
} );
// These tests are unreliable in Firefox
if ( !( /firefox/i.test( window.navigator.userAgent ) ) ) {
QUnit.test( "Check order of focusin/focusout events", function( assert ) {

0 comments on commit 67fa2ea

Please sign in to comment.