From 4deacf11d64e2c34caa4fd87c7461c307795be3b Mon Sep 17 00:00:00 2001
From: Joseph Schorr <joseph.schorr@coreos.com>
Date: Thu, 23 Apr 2015 12:12:54 -0400
Subject: [PATCH] Entity search menu improvements: sorting and max height

---
 static/css/directives/ui/entity-search.css | 6 ++++++
 static/directives/entity-search.html       | 5 +++--
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/static/css/directives/ui/entity-search.css b/static/css/directives/ui/entity-search.css
index eaab63120..dcc5d6ab6 100644
--- a/static/css/directives/ui/entity-search.css
+++ b/static/css/directives/ui/entity-search.css
@@ -40,6 +40,12 @@
   margin-right: 36px;
 }
 
+.entity-search-element .entity-menu {
+  overflow-x: hidden;
+  overflow-y: auto;
+  max-height: 400px;
+}
+
 .entity-search-element .dropdown {
   vertical-align: middle;
   position: absolute;
diff --git a/static/directives/entity-search.html b/static/directives/entity-search.html
index ae181735d..9e8742408 100644
--- a/static/directives/entity-search.html
+++ b/static/directives/entity-search.html
@@ -5,7 +5,8 @@
             ng-click="lazyLoad()">
       <span class="caret"></span>
     </button>
-    <ul class="dropdown-menu" ng-class="pullRight == 'true' ? 'pull-right': ''" role="menu" aria-labelledby="entityDropdownMenu">
+    <ul class="dropdown-menu entity-menu" ng-class="pullRight == 'true' ? 'pull-right': ''" role="menu"
+        aria-labelledby="entityDropdownMenu">
       <li ng-show="lazyLoading" style="padding: 10px"><div class="cor-loader"></div></li>
 
       <li role="presentation" class="dropdown-header" ng-show="!lazyLoading && !robots && !isAdmin && !teams">
@@ -67,7 +68,7 @@
 
       <li role="presentation" class="dropdown-header" ng-show="!lazyLoading && robots">Robot Accounts</li>
 
-      <li class="menuitem" role="presentation" ng-repeat="robot in robots" ng-show="!lazyLoading">
+      <li class="menuitem" role="presentation" ng-repeat="robot in robots | orderBy:name" ng-show="!lazyLoading">
         <a role="menuitem" tabindex="-1" href="javascript:void(0)" ng-click="setEntity(robot.name, 'user', true)">
           <i class="fa fa-wrench"></i> <span>{{ robot.name }}</span>
         </a>