Enterprise: Application Menu is unexpectedly closing when clicking headers in the mobile breakpoint

Created on 30 Jul 2018 · 6Comments · Source: infor-design/enterprise

Originally reported by @vonnyw

Describe the bug
When an application menu is open below the mobile breakpoint threshold, and one of its accordion headers is clicked, the menu is closing unexpectedly -- even if the dismissOnClickMobile setting is set to false.

To Reproduce
Steps to reproduce the behavior:

Go to http://master-enterprise.demo.design.infor.com/components/applicationmenu/example-container.html
Resize the browser viewport to fit beneath the Mobile breakpoint (767px or less)
Open the Application Menu using the hamburger
Click/Tap on any of the accordion headers
Also clicking plus and minus icons should not close the menu

The Application menu is unexpectedly closing.

Expected behavior
The Application menu should remain open.

Videos
See the webm file here for a video displaying the issue:
application-menu-close-premature.webm.zip

Platform(s) n/a

Additional context
After some troubleshooting we discovered that the first condition on [this line] (https://github.com/infor-design/enterprise/blob/master/src/components/applicationmenu/applicationmenu.js#L430) in the App Menu is showing e.target being the body tag. This value should be represented as an a or .accordion-header element, and should not the body tag. Possible things to check:

pointer-events: none; CSS rule on something inside the app menu where it shouldn't be?

[3] critical type

Source

EdwardCoyle

All 6 comments

I'm also running into this same issue while conducting smoke testing.

CindyMercadoReyes on 30 Jul 2018

👍1

@clepore @theakatz due to the critical tag, should we add this one to the July sprint for fixing? I might be able to track it down and get it fixed today.

EdwardCoyle on 31 Jul 2018

Yes we think so