Merge pull request #8 from arnoudhgz/bugfix/13327-ui-active-state-not-removed-from-previous-menu-item

arnoudhgz · web-flow · commit 5773225311c8 · 2018-01-24T07:22:59.000+01:00
Remove ui-state-active from siblings of the expanded menu item
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`