From fcd29239535b1579e5dbdfdf452a98a4bfc1bb13 Mon Sep 17 00:00:00 2001
From: ryan <ryan@1a063a9b-81f0-0310-95a4-ce76da25c4cd>
Date: Mon, 4 Jun 2012 14:50:50 +0000
Subject: [PATCH] Site/user autocomplete fixes.

* Lengthen the inputs in the Right Now network dashboard widget
* Border around the whole autocomplete widget/list
* Handle long strings
* Remove some CSS color cruft from the main stylesheet and de-dupe the JS closures for open/close callbacks.

Props helenyhou
fixes #20584


git-svn-id: http://core.svn.wordpress.org/trunk@20984 1a063a9b-81f0-0310-95a4-ce76da25c4cd
---
 wp-admin/css/wp-admin.dev.css   |  7 -------
 wp-admin/includes/dashboard.php |  4 ++--
 wp-admin/js/site-search.dev.js  | 10 +++++----
 wp-admin/js/user-search.dev.js  | 37 +++++++++++++++++++++------------
 4 files changed, 32 insertions(+), 26 deletions(-)

diff --git a/wp-admin/css/wp-admin.dev.css b/wp-admin/css/wp-admin.dev.css
index 4eed67961f..35297c15a6 100644
--- a/wp-admin/css/wp-admin.dev.css
+++ b/wp-admin/css/wp-admin.dev.css
@@ -889,28 +889,21 @@ ul#add-to-blog-users {
 	border-bottom-left-radius: 3px;
 	border-width: 1px;
 	border-style: solid;
-	border-top: none;
-
-	background-color: #efefef;
-	border-color: #bbb;
 }
 
 .ui-autocomplete li {
 	margin-bottom: 0;
 	white-space: nowrap;
 	text-align: left;
-	color: #101010;
 }
 
 .ui-autocomplete li a {
 	display: block;
 	height: 100%;
 	padding: 4px 10px;
-	color: #333;
 }
 
 .ui-autocomplete li a.ui-state-hover {
-	background-color: #ddd;
 	cursor: pointer;
 }
 
diff --git a/wp-admin/includes/dashboard.php b/wp-admin/includes/dashboard.php
index 37408e1287..b284f05dc2 100644
--- a/wp-admin/includes/dashboard.php
+++ b/wp-admin/includes/dashboard.php
@@ -444,14 +444,14 @@ function wp_network_dashboard_right_now() {
 
 	<form name="searchform" action="<?php echo network_admin_url('users.php'); ?>" method="get">
 		<p>
-			<input type="search" name="s" value="" size="17" id="all-user-search-input" />
+			<input type="search" name="s" value="" size="30" id="all-user-search-input" />
 			<?php submit_button( __( 'Search Users' ), 'button', 'submit', false, array( 'id' => 'submit_users' ) ); ?>
 		</p>
 	</form>
 
 	<form name="searchform" action="<?php echo network_admin_url('sites.php'); ?>" method="get">
 		<p>
-			<input type="search" name="s" value="" size="17" id="site-search-input" />
+			<input type="search" name="s" value="" size="30" id="site-search-input" />
 			<?php submit_button( __( 'Search Sites' ), 'button', 'submit', false, array( 'id' => 'submit_sites' ) ); ?>
 		</p>
 	</form>
diff --git a/wp-admin/js/site-search.dev.js b/wp-admin/js/site-search.dev.js
index 8c89410887..e47d7a4c99 100644
--- a/wp-admin/js/site-search.dev.js
+++ b/wp-admin/js/site-search.dev.js
@@ -1,9 +1,11 @@
 jQuery( function($) {
+	var isRTL = !! ( 'undefined' != typeof isRtl && isRtl );
 	$( '#site-search-input' ).autocomplete({
-		source:   ajaxurl + '?action=autocomplete-site',
-		delay:    500,
+		source:    ajaxurl + '?action=autocomplete-site',
+		delay:     500,
 		minLength: 2,
-		open: function(e, ui) { $(this).addClass('open'); },
-		close: function(e, ui) { $(this).removeClass('open'); }
+		position:  isRTL ? { my: 'right top', at: 'right bottom', offset: '0, -1' } : { offset: '0, -1' },
+		open:      function(e, ui) { $(this).addClass('open'); },
+		close:     function(e, ui) { $(this).removeClass('open'); }
 	});
 });
diff --git a/wp-admin/js/user-search.dev.js b/wp-admin/js/user-search.dev.js
index e8d7fec16f..ce3b63a799 100644
--- a/wp-admin/js/user-search.dev.js
+++ b/wp-admin/js/user-search.dev.js
@@ -1,27 +1,38 @@
 jQuery( function($) {
-	var id = typeof( current_site_id ) != 'undefined' ? '&site_id=' + current_site_id : '';
+	var id = typeof( current_site_id ) != 'undefined' ? '&site_id=' + current_site_id : '',
+	    isRTL = !! ( 'undefined' != typeof isRtl && isRtl ),
+	    position = isRTL ? { my: 'right top', at: 'right bottom', offset: '0, -1' } : { offset: '0, -1' },
+	    open = function(e, ui) {
+	    	$(this).addClass('open');
+	    },
+	    close = function(e, ui) {
+	    	$(this).removeClass('open');
+	    };
 
 	$( '#adduser-email, #newuser' ).autocomplete({
-		source:   ajaxurl + '?action=autocomplete-user&autocomplete_type=add' + id,
-		delay:    500,
+		source:    ajaxurl + '?action=autocomplete-user&autocomplete_type=add' + id,
+		delay:     500,
 		minLength: 2,
-		open: function(e, ui) { $(this).addClass('open'); },
-		close: function(e, ui) { $(this).removeClass('open'); }
+		position:  position,
+		open:      open,
+		close:     close
 	});
 
 	$( '#user-search-input' ).autocomplete({
-		source:   ajaxurl + '?action=autocomplete-user&autocomplete_type=search' + id,
-		delay:    500,
+		source:    ajaxurl + '?action=autocomplete-user&autocomplete_type=search' + id,
+		delay:     500,
 		minLength: 2,
-		open: function(e, ui) { $(this).addClass('open'); },
-		close: function(e, ui) { $(this).removeClass('open'); }
+		position:  position,
+		open:      open,
+		close:     close
 	});
 
 	$( '#all-user-search-input' ).autocomplete({
-		source:   ajaxurl + '?action=autocomplete-user&autocomplete_type=search-all' + id,
-		delay:    500,
+		source:    ajaxurl + '?action=autocomplete-user&autocomplete_type=search-all' + id,
+		delay:     500,
 		minLength: 2,
-		open: function(e, ui) { $(this).addClass('open'); },
-		close: function(e, ui) { $(this).removeClass('open'); }
+		position:  position,
+		open:      open,
+		close:     close
 	});
 });