From c074006a69db73a116dc04ec78844468a8cea7d3 Mon Sep 17 00:00:00 2001 From: Oleg Gaidarenko Date: Wed, 13 May 2015 01:15:19 +0300 Subject: Event: provide verbose comment for focus(in | out) & rename support prop Closes gh-2312 --- src/event.js | 16 +++++++++++----- src/event/support.js | 2 +- 2 files changed, 12 insertions(+), 6 deletions(-) (limited to 'src') diff --git a/src/event.js b/src/event.js index 584ba9073..361a5489f 100644 --- a/src/event.js +++ b/src/event.js @@ -748,15 +748,21 @@ jQuery.each({ }; }); -// Support: Firefox, Chrome, Safari -// Create "bubbling" focus and blur events -if ( !support.focusinBubbles ) { +// Support: Firefox +// Firefox doesn't have focus(in | out) events +// Related ticket - https://bugzilla.mozilla.org/show_bug.cgi?id=687787 +// +// Support: Chrome, Safari +// focus(in | out) events fire after focus & blur events, +// which is spec violation - http://www.w3.org/TR/DOM-Level-3-Events/#events-focusevent-event-order +// Related ticket - https://code.google.com/p/chromium/issues/detail?id=449857 +if ( !support.focusin ) { jQuery.each({ focus: "focusin", blur: "focusout" }, function( orig, fix ) { // Attach a single capturing handler on the document while someone wants focusin/focusout var handler = function( event ) { - jQuery.event.simulate( fix, event.target, jQuery.event.fix( event ), true ); - }; + jQuery.event.simulate( fix, event.target, jQuery.event.fix( event ), true ); + }; jQuery.event.special[ fix ] = { setup: function() { diff --git a/src/event/support.js b/src/event/support.js index 85060db77..7ddca6a8d 100644 --- a/src/event/support.js +++ b/src/event/support.js @@ -2,7 +2,7 @@ define([ "../var/support" ], function( support ) { -support.focusinBubbles = "onfocusin" in window; +support.focusin = "onfocusin" in window; return support; -- cgit v1.2.3