search

mozilla / sumo-experiments

1 stars 5 forks source link

[mobile] The opened menu is displayed under the grayed out menu options #52

Open rleitan opened 5 years ago

rleitan commented 5 years ago

[Environment]: iOS, Android - Firefox, Safari, Chrome

[Steps to reproduce]: 1.Open https://support.allizom.org/en-US/kb/exp/insecure-password-warning-firefox 2.Tap on one of the nav menu option (e.g. Cookies and cache)

[Expected result]: The menu should be opened and cover the other menu options ( like on web- see attached photo)

2019-02-25_13h17_53 [Actual result]: The other menu options are still visible(grayed out) and the opened menu is displayed under grayed out menu options

image1

akatsoulas commented 5 years ago

@rleitan Can you please elaborate why this is a bug? This is happening only when you expand the Cookies and Cache submenu and it showcases what's active and what's not. When everything is collapsed the Insecure password warning in Firefox is shown with bold.

rleitan commented 5 years ago

I agree it's clear what is active and what is not. But I think should be consistent with the desktop behaviour, the inactive menu option (greyed out on mobile) should be overlaid by the active ones, so the menu will not be displayed in addition to the inactive menu options

akatsoulas commented 5 years ago

I would defer this to @madasan as it's more of a UX matter and not an actual bug. We will treat it as such if we want the same behavior as in desktop.