-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Prevent remove click listener when switch from mobile to desktop mode. #10430
Conversation
Prevent remove click listener when switch from mobile to desktop mode.
lib/web/mage/menu.js
Outdated
@@ -371,7 +371,8 @@ define([ | |||
*/ | |||
_toggleDesktopMode: function () { | |||
var categoryParent, html; | |||
|
|||
|
|||
$(this.element).off('click mouseenter mouseleave'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please, add also mousedown.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added mousedown as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, thanks.
…to desktop mode. - Fixed test failures regarding coding style
@magento-team Will this be backported to 2.2? |
Description
When clicking on level0 menu item after switching from mobile to desktop mode the click event is not active. This only occurs for menu items with children.
Manual testing scenarios
Contribution checklist