Autocomplete: Trigger close after hiding the menu. Fixes #6227 - Auto…

…complete: Don't trigger close event until after the menu is hidden.
jquery · Oct 26, 2010 · f4d8ec9 · f4d8ec9
1 parent 9ead629
commit f4d8ec9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/tests/unit/autocomplete/autocomplete_events.js b/tests/unit/autocomplete/autocomplete_events.js
@@ -28,7 +28,7 @@ test("all events", function() {
 		},
 		close: function(event) {
 			same(event.type, "autocompleteclose");
-			same( $(".ui-menu:visible").length, 1 );
+			same( $(".ui-menu:visible").length, 0 );
 		},
 		select: function(event, ui) {
 			same(event.type, "autocompleteselect");

diff --git a/ui/jquery.ui.autocomplete.js b/ui/jquery.ui.autocomplete.js
@@ -295,9 +295,9 @@ $.widget( "ui.autocomplete", {
 	close: function( event ) {
 		clearTimeout( this.closing );
 		if ( this.menu.element.is(":visible") ) {
-			this._trigger( "close", event );
 			this.menu.element.hide();
 			this.menu.deactivate();
+			this._trigger( "close", event );
 		}
 	},