hd-zero/hdzero-goggle

Menu highlight should change when in sub-menu

Closed this issue · 5 comments

evilC commented

When on the main menu, when you roll the wheel, the selected entry in the main menu is highlighted grey to indicate that the current sub-menu is selected.
When you click to enter that sub-menu, the highlight of the main menu remains grey and the sub-menu also has a highlight, which is also grey.
I feel that the main menu highlight should change in some way (Maybe turn green?) to indicate this state.

All too frequently I find myself accidentally long pressing once too many times and exiting to the FPV view, and I think this is why

I agree... there needs to be some visual indication that you are in the subpage, and not interacting with the sidebar menu still. It could even be as simple as the red arrow NOT being shown as you scroll, and only being visible once you press on a menu item.

I fiddled a little with the menu design and came up with the following proposal:

screenshot_menu_highlight

It changes the highlighting color to a lighter grey and removes the red arrow as soon as you enter a submenu.
What do you think about that?

I tend to think it makes sense for the arrow to be not shown until you enter the submenu/page, but I guess using it will tell if that makes sense or not. PR when? 🤣

I see what you mean and thought the same. However the red arrow unfortunately doesn't have an alpha channel so when I tried to do that, there was an awkward grey-ish box around it. I think, we'd need to update the resource then but unfortunately, the original icon image is nowhere to be found 😣

You mean something like this?

screenshot_menu_highlight_v2

Do you really think I should open a PR? I mean, this pretty significantly impacts the UX...