Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove ui-state-active from siblings of the expanded menu item #8

Merged
merged 1 commit into from
Jan 24, 2018
Merged

Remove ui-state-active from siblings of the expanded menu item #8

merged 1 commit into from
Jan 24, 2018

Conversation

arnoudhgz
Copy link

Description

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.

Fixed Issues (if relevant)

  1. Menu ui-state-active not removed from previous opened menu item magento/magento2#13327 Menu ui-state-active not removed from previous opened menu item

Manual testing scenarios

  1. In your browser switch to mobile view so you have the mobile menu
  2. Open the menu
  3. Expand the menu item Women
  4. Now expand the menu item Men
  5. See that the ui-state-active class is removed from Women

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

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.
@arnoudhgz arnoudhgz changed the base branch from 2.2-develop to bugfix/13327-ui-active-state-not-removed-from-previous-menu-item January 23, 2018 12:41
@arnoudhgz arnoudhgz requested review from jitze and ksangers January 23, 2018 12:41
@okorshenko
Copy link

Hi
Could you please create PR to magento/magento2 repository?

@arnoudhgz arnoudhgz merged commit 5773225 into mediact:bugfix/13327-ui-active-state-not-removed-from-previous-menu-item Jan 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants