Merge forwardport of #13341 to 2.3-develop branch

magento-engcom-team · web-flow · commit 61d0d9608493 · 2018-02-15T10:20:43.000-06:00
Applied pull request patch https://github.com/magento/magento2/pull/13341.patch (created by @arnoudhgz) based on commit(s): 1. 5714ffd 2. 5773225 Fixed GitHub Issues in 2.3-develop branch: - #13327: Menu ui-state-active not removed from previous opened menu item (reported by @arnoudhgz)
diff --git a/lib/web/mage/menu.js b/lib/web/mage/menu.js
@@ -629,6 +629,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
`@@ -629,6 +629,9 @@ define([`
`629`	`629`	`return;`
`630`	`630`	`}`
`631`	`631`
	`632`	`+ // remove the active state class from the siblings`
	`633`	`+ this.active.siblings().children('.ui-state-active').removeClass('ui-state-active');`
	`634`	`+`
`632`	`635`	`this._open(newItem.parent());`
`633`	`636`
`634`	`637`	`// Delay so Firefox will not hide activedescendant change in expanding submenu from AT`