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

Oleksii Korshenko · Oleksii Korshenko · commit efa1251648ba · 2018-01-29T09:54:13.000-06:00
- Merge Pull Request magento#13341 from mediact/magento2:bugfix/13327-ui-active-state-not-removed-from-previous-menu-item - Merged commits: 1. 5714ffd 2. 5773225
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`