633656 - When split menus are hovered, both the main item and expansion arrow should display on mouse in

Status: RESOLVED FIXED

(Firefox :: Menus, defect)

Description

[Alex Faaborg [:faaborg] (Firefox UX)]

When split menus are hovered, both the main item and expansion arrow should display simultaneously on mouse in (of either item).

Currently the main item hover effect displays on mouse in, and then the expansion arrow hover effect displays on a timer delay.

The windows 7 start menu can be used as a reference for the behavior.  Note that this bug just covers displaying the hover effect (in the current style).  Additional styling work is covered over in bug 628049.

Comment 1

[Alex Limi (:limi) — Firefox UX Team]

I'd like to nominate this for blocking final. 

The reason is that the new menu is one of the first things people encounter in Firefox 4, and the current hover behavior makes it feel unstable and weird, since it doesn't match the hover and display behaviors elsewhere in the OS.

Comment 2

[Alex Limi (:limi) — Firefox UX Team]

The other half of this fix is bug 628048.

Comment 3

[Dietrich Ayala (:dietrich)]

Unstable is pretty strong for describing this. Weird... yes, a bit. The menu does display, just after a slight delay, so not blocking on this. It's a nice bit of interaction polish, but we can't keep shipping Firefox 3.6 just for this change. blocking- and wanted+.

Comment 4

[:Margaret Leibovic]

Attachment: patch

Comment 5

[Dão Gottwald [:dao]]

Comment on attachment 512514 [details] [diff] [review]
patch

Along with this I think we need a 1px margin between the two parts, otherwise they won't be distinguishable if the theme draws a solid background for active items.

Comment 6

[:Margaret Leibovic]

Attachment: patch v2

Added 1px between two parts of split menu. I didn't know if the styles should be in content or theme, so I just put them in theme.

Comment 7

[Alex Faaborg [:faaborg] (Firefox UX)]

>Added 1px between two parts of split menu. I didn't know if the styles should
>be in content or theme, so I just put them in theme.

Thanks, I was actually going to request that as well :)

Comment 8

[Dão Gottwald [:dao]]

Comment on attachment 512568 [details] [diff] [review]
patch v2

>--- a/browser/themes/gnomestripe/browser/browser.css
>+++ b/browser/themes/gnomestripe/browser/browser.css

>+.splitmenu-menuitem {
>+  -moz-margin-end: 1px;
>+}
>+
> /* Application menu toolbar button */

Please move the comment up and change it to /* Application button menu */.
r=me with that

Comment 9

[:Margaret Leibovic]

http://hg.mozilla.org/mozilla-central/rev/45895e0903f2