MAGETWO-87242: Bugfix/13327 ui active state not removed from previous menu item #13341

Oleksii Korshenko · web-flow · commit 03b9e164cdbe · 2018-01-30T16:35:06.000-06:00
diff --git 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

Original file line number	Diff line number	Diff line change
`@@ -626,6 +626,9 @@ define([`
`626`	`626`	`return;`
`627`	`627`	`}`
`628`	`628`
	`629`	`+ // remove the active state class from the siblings`
	`630`	`+ this.active.siblings().children('.ui-state-active').removeClass('ui-state-active');`
	`631`	`+`
`629`	`632`	`this._open(newItem.parent());`
`630`	`633`
`631`	`634`	`// Delay so Firefox will not hide activedescendant change in expanding submenu from AT`