From: Scott González <scott.gonzalez@gmail.com>
Date: Mon, 19 Jul 2010 18:36:40 +0000 (-0400)
Subject: Autocomplete: Don't update the value of the text field until after setting focus... 
X-Git-Tag: 1.8.3~38^2~3
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=53489b502dc51edb7707de80ead77549bd529e3a;p=jquery-ui.git

Autocomplete: Don't update the value of the text field until after setting focus. Fixes #5639 - Cursor jumps to beginning on select.
---

diff --git a/ui/jquery.ui.autocomplete.js b/ui/jquery.ui.autocomplete.js
index 3f010c1f4..c25a8884f 100644
--- a/ui/jquery.ui.autocomplete.js
+++ b/ui/jquery.ui.autocomplete.js
@@ -132,9 +132,12 @@ $.widget( "ui.autocomplete", {
 					}
 				},
 				selected: function( event, ui ) {
-					var item = ui.item.data( "item.autocomplete" );
+					var item = ui.item.data( "item.autocomplete" ),
+						setValue = false;
 					if ( false !== self._trigger( "select", event, { item: item } ) ) {
-						self.element.val( item.value );
+						// #5639 - if we set the value before setting focus
+						// the cursor will move to the beginning of the field in IE
+						setValue = true;
 					}
 					self.close( event );
 					// only trigger when focus was lost (click on menu)
@@ -144,6 +147,9 @@ $.widget( "ui.autocomplete", {
 						self.previous = previous;
 					}
 					self.selectedItem = item;
+					if ( setValue ) {
+						self.element.val( item.value );
+					}
 				},
 				blur: function( event, ui ) {
 					if ( self.menu.element.is(":visible") ) {