Skip to content
This repository
Browse code

[popup] Do not fiddle with the focus when rapid-opening/closing durin…

…g reposition - Fixes: #4797 (hopefully)
  • Loading branch information...
commit 5680ba620c719de7ea4c2c1ec97928f58442e8a9 1 parent b2c66d6
Gabriel "_|Nix|_" Schulhof authored August 12, 2012

Showing 1 changed file with 2 additions and 5 deletions. Show diff stats Hide diff stats

  1. 7  js/widgets/popup.js
7  js/widgets/popup.js
@@ -91,9 +91,7 @@ define( [ "jquery",
91 91
 				this._trigger( "beforeposition" );
92 92
 				this._ui.container
93 93
 					.removeClass( "ui-selectmenu-hidden" )
94  
-					.offset( this._placementCoords( this._desiredCoords( undefined, undefined, "window" ) ) )
95  
-					.attr( "tabindex", "0" )
96  
-					.focus();
  94
+					.offset( this._placementCoords( this._desiredCoords( undefined, undefined, "window" ) ) );
97 95
 
98 96
 				this._resizeData = null;
99 97
 				this._orientationchangeInProgress = false;
@@ -112,8 +110,7 @@ define( [ "jquery",
112 110
 				// effectively rapid-close the popup while leaving the screen intact
113 111
 				this._ui.container
114 112
 					.addClass( "ui-selectmenu-hidden" )
115  
-					.removeAttr( "style" )
116  
-					.removeAttr( "tabindex" );
  113
+					.removeAttr( "style" );
117 114
 
118 115
 				this._orientationchangeInProgress = true;
119 116
 			}

0 notes on commit 5680ba6

Please sign in to comment.
Something went wrong with that request. Please try again.