Merge pull request #23609 from briennekordis/leading_zero_fix

[civicrm-core.git] / js / crm.menubar.js

diff --git a/js/crm.menubar.js b/js/crm.menubar.js
index a9177bb5b2ced16a5bcff09953e02aa41d9acc2c..c75e38c5e5ba0f1e711433ebc4a747846d0f277d 100644 (file)
--- a/js/crm.menubar.js
+++ b/js/crm.menubar.js
@@ -314,6 +314,12 @@
             });
           },
           focus: function (event, ui) {
+            // This is when an item is 'focussed' by keyboard up/down or mouse hover.
+            // It is not the same as actually having focus, i.e. it is not :focus
+            var lis = $(event.currentTarget).find('li[data-cid="' + ui.item.value + '"]');
+            lis.children('div').addClass('ui-state-active');
+            lis.siblings().children('div').removeClass('ui-state-active');
+            // Returning false leaves the user-entered text as it was.
             return false;
           },
           select: function (event, ui) {
@@ -340,6 +346,7 @@
         .autocomplete( "instance" )._renderItem = function( ul, item ) {
           var uiMenuItemWrapper = $("<div class='ui-menu-item-uiMenuItemWrapper'>");
           if (item.value == 0) {
+            // "No results"
             uiMenuItemWrapper.text(item.label);
           }
           else {
@@ -359,7 +366,7 @@
               }));
           }
 
-          return $( "<li class='ui-menu-item'>" )
+          return $( "<li class='ui-menu-item' data-cid=" + item.value + ">" )
             .append(uiMenuItemWrapper)
             .appendTo( ul );
         };