[wp-trac] [WordPress Trac] #29906: Submenus can't be dismissed on mobile.

[WordPress Trac]

#29906: Submenus can't be dismissed on mobile.
---------------------------------+------------------------------
 Reporter:  obenland             |       Owner:
     Type:  defect (bug)         |      Status:  new
 Priority:  normal               |   Milestone:  Awaiting Review
Component:  Toolbar              |     Version:
 Severity:  normal               |  Resolution:
 Keywords:  make-flow has-patch  |     Focuses:
---------------------------------+------------------------------
Changes (by dancameron):

 * keywords:  make-flow => make-flow has-patch


Comment:

 When working on this I noticed that the `.ab-sub-wrapper` is set to
 `position: fixed;`, this will cause the menu to essentially take over any
 small screens and on somewhat larger screens (see example above) the menu
 would stick and break away from the header menu. My patch attached will
 set it to  `position: absolute;`, which will fix the weirdness but will
 also provide a way for small screens to get out of the drop-down submenu
 (e.g. scroll down and touch the content).

 Something not necessary in the patch is to disable the dash-icon that
 links to the dashboard on these smaller screens. Seems redundant and
 unnecessary and if the user expects the icon to close the drop-down
 they're now not unexpectedly sent to the admin.

 I wanted to ask two things before proceeding:

 1. Should we disable that icon?
 2. Should we go further and allow for that icon to close the submenu on
 smaller screens and/or enable a touch to the other areas of the admin bar
 close it?

 IMO # 1 is warranted and a UX argument could be had. As for # 2 I'm mixed
 since IMO the current patch fixes the UX and UI bug.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/29906#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform