From 5714ffd97f8b8f21c46e047d74a02bd75742002b Mon Sep 17 00:00:00 2001 From: Arnoud Beekman Date: Tue, 23 Jan 2018 13:32:55 +0100 Subject: [PATCH] Remove ui-state-active from siblings of the expanded menu item Previously when a menu item was expanded the class `ui-state-active` was not removed from the previous expanded menu item. This resulted in two (or more if you expanded more) menu items with this class. --- lib/web/mage/menu.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lib/web/mage/menu.js b/lib/web/mage/menu.js index 6ad60333d2e1b..41730aa036b08 100644 --- a/lib/web/mage/menu.js +++ b/lib/web/mage/menu.js @@ -626,6 +626,9 @@ define([ return; } + // remove the active state class from the siblings + this.active.siblings().children('.ui-state-active').removeClass('ui-state-active'); + this._open(newItem.parent()); // Delay so Firefox will not hide activedescendant change in expanding submenu from AT