From bc6acee1f1ebbdd7ad6fea460f48125f3b20b7cd Mon Sep 17 00:00:00 2001
From: Denis Flaven <denis.flaven@combodo.com>
Date: Sat, 12 Sep 2015 09:14:39 +0000
Subject: [PATCH] Cosmetics on the "autocomplete": more compact by default (20
 chars instead of 30), and buttons evenly spaced.

SVN:trunk[3750]
---
 application/ui.extkeywidget.class.inc.php | 4 ++--
 css/jquery.autocomplete.css               | 2 ++
 js/jquery.autocomplete.js                 | 4 ++--
 3 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/application/ui.extkeywidget.class.inc.php b/application/ui.extkeywidget.class.inc.php
index d61e42308..c5dac3ae5 100644
--- a/application/ui.extkeywidget.class.inc.php
+++ b/application/ui.extkeywidget.class.inc.php
@@ -253,11 +253,11 @@ EOF
 				$sDisplayValue = $this->GetObjectName($value);
 			}
 			$iMinChars = isset($aArgs['iMinChars']) ? $aArgs['iMinChars'] : 3; //@@@ $this->oAttDef->GetMinAutoCompleteChars();
-			$iFieldSize = isset($aArgs['iFieldSize']) ? $aArgs['iFieldSize'] : 30; //@@@ $this->oAttDef->GetMaxSize();
+			$iFieldSize = isset($aArgs['iFieldSize']) ? $aArgs['iFieldSize'] : 20; //@@@ $this->oAttDef->GetMaxSize();
 	
 			// the input for the auto-complete
 			$sHTMLValue = "<input count=\"".$oAllowedValues->Count()."\" type=\"text\" id=\"label_$this->iId\" size=\"$iFieldSize\" value=\"$sDisplayValue\"/>&nbsp;";
-			$sHTMLValue .= "<img id=\"mini_search_{$this->iId}\" style=\"border:0;vertical-align:middle;cursor:pointer;\" src=\"../images/mini_search.gif\" onClick=\"oACWidget_{$this->iId}.Search();\"/>&nbsp;";
+			$sHTMLValue .= "<img id=\"mini_search_{$this->iId}\" style=\"border:0;vertical-align:middle;cursor:pointer;\" src=\"../images/mini_search.gif\" onClick=\"oACWidget_{$this->iId}.Search();\"/>";
 	
 			// another hidden input to store & pass the object's Id
 			$sHTMLValue .= "<input type=\"hidden\" id=\"$this->iId\" name=\"{$sAttrFieldPrefix}{$sFieldName}\" value=\"".htmlentities($value, ENT_QUOTES, 'UTF-8')."\" />\n";
diff --git a/css/jquery.autocomplete.css b/css/jquery.autocomplete.css
index 4e6b634f1..f089a2ade 100644
--- a/css/jquery.autocomplete.css
+++ b/css/jquery.autocomplete.css
@@ -17,6 +17,8 @@
 .ac_results li {
 	margin: 0px;
 	padding: 2px 5px;
+	white-space: nowrap;
+	padding-right: 20px; /* Space for the scrollbar */ 
 	cursor: default;
 	display: block;
 	/* 
diff --git a/js/jquery.autocomplete.js b/js/jquery.autocomplete.js
index 0b9fc7027..a38d71be9 100644
--- a/js/jquery.autocomplete.js
+++ b/js/jquery.autocomplete.js
@@ -671,7 +671,7 @@ $.Autocompleter.Select = function (options, input, select, config) {
 				continue;
 			// Escape dangerous characters to prevent XSS vulnerabilities
 			formatted = formatted.replace('&', '&amp;').replace('"', '&quot;').replace('>', '&gt;').replace('<', '&lt;');
-			var li = $("<li/>").html( options.highlight(formatted, term) ).addClass(i%2 == 0 ? "ac_even" : "ac_odd").appendTo(list)[0];
+			var li = $("<li/>").html( options.highlight(formatted, term) ).addClass('ac_item').addClass(i%2 == 0 ? "ac_even" : "ac_odd").appendTo(list)[0];
 			$.data(li, "ac_data", data[i]);
 		}
 		listItems = list.find("li");
@@ -725,7 +725,7 @@ $.Autocompleter.Select = function (options, input, select, config) {
 		show: function() {
 			var offset = $(input).offset();
 			element.css({
-				width: typeof options.width == "string" || options.width > 0 ? options.width : $(input).width(),
+				'min-width': typeof options.width == "string" || options.width > 0 ? options.width : $(input).width(),
 				top: offset.top + input.offsetHeight,
 				left: offset.left
 			}).show();