Merge pull request #7989 from adobe/rlim/dropdown-handler-cleanup

Fix filter dropdown list highlighting issue with re-registering mouse event handlers.

Author: bchintx

src/utils/DropdownEventHandler.js

@@ -293,6 +293,21 @@ define(function (require, exports, module) {
                 }
             });
     };
+    
+    /**
+     * Re-register mouse event handlers
+     * @param {!jQueryObject} $list  newly updated list object
+     */
+    DropdownEventHandler.prototype.reRegisterMouseHandlers = function ($list) {
+        if (this.$list) {
+            this.$list.off("click mouseover");
+            
+            this.$list = $list;
+            this.$items = $list.find("li");
+            
+            this._registerMouseEvents();
+        }
+    };
 
     // Export public API
     exports.DropdownEventHandler    = DropdownEventHandler;

src/widgets/DropdownButton.js

@@ -150,10 +150,9 @@ define(function (require, exports, module) {
         // set up for any custom UI in the list.
         $(this).triggerHandler("listRendered", [parent]);
 
+        // Also need to re-register mouse event handlers with the updated list.
         if (this._dropdownEventHandler) {
-            // Re-create and re-attach event handlers for the new rendered list.
-            this._dropdownEventHandler = new DropdownEventHandler(parent, this._onSelect.bind(this), this._onDropdownClose.bind(this));
-            this._dropdownEventHandler.open();
+            this._dropdownEventHandler.reRegisterMouseHandlers(parent);
         }
 
         return parent;