1408450 - Implement doorhanger sub-menu animation and new icon

Status: NEW

(Firefox :: Menus, enhancement, P4)

Description

[Amy Lee [:amylee] UX]

In response to bug 1390551. This is the proposed animation to make it clearer to users that you need to click on the menu item to access the sub-menu.

I've attached svg icon assets. Let me know if this requires a sprite sheet.

Comment 1

[Amy Lee [:amylee] UX]

Attachment: Options.svg

Comment 2

[Amy Lee [:amylee] UX]

Attachment: Arrow.svg

Comment 3

[Jared Wein [:jaws] (please needinfo? me)]

We would need a sprite sheet to implement the animation tweening that is showing in https://bug1390551.bmoattachments.org/attachment.cgi?id=8917870

But I would prefer that we revisit the choice to not expand on hover. Have we detailed our reasoning for not supporting expand on hover? Is it because we change subviews in place instead of expanding the menu outwards?

Comment 4

[Amy Lee [:amylee] UX]

(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #3)
> We would need a sprite sheet to implement the animation tweening that is
> showing in https://bug1390551.bmoattachments.org/attachment.cgi?id=8917870
> 
> But I would prefer that we revisit the choice to not expand on hover. Have
> we detailed our reasoning for not supporting expand on hover? Is it because
> we change subviews in place instead of expanding the menu outwards?

I believe the reasoning is that a user can accidentally switch to the sub-menu on hover. In the old version the menu expands out so it's not as big of a deal but it could be more frustrating in this case if the user didn't intend on going to the sub-menu and need to click out of it. NI Stephen, see Jared's comment above.

Comment 5

[Stephen Horlander [:shorlander]]

(In reply to Amy Lee [:amylee] UX from comment #4)
> (In reply to Jared Wein [:jaws] (please needinfo? me) from comment #3)
> > We would need a sprite sheet to implement the animation tweening that is
> > showing in https://bug1390551.bmoattachments.org/attachment.cgi?id=8917870
> > 
> > But I would prefer that we revisit the choice to not expand on hover. Have
> > we detailed our reasoning for not supporting expand on hover? Is it because
> > we change subviews in place instead of expanding the menu outwards?
> 
> I believe the reasoning is that a user can accidentally switch to the
> sub-menu on hover. In the old version the menu expands out so it's not as
> big of a deal but it could be more frustrating in this case if the user
> didn't intend on going to the sub-menu and need to click out of it. NI
> Stephen, see Jared's comment above.

Yes. It's awkward and confusing to have the UI rearrange itself on hover. Sub-menus don't have that problem because they appear next to (or over) what you are hovering.

Comment 6

[Jared Wein [:jaws] (please needinfo? me)]

Amy, would you be able to supply the SVG sprite?

Comment 7

[Amy Lee [:amylee] UX]

Attachment: submenu_icon_animation.svg

Here's the sprite

Comment 8

[Amy Lee [:amylee] UX]

Attachment: submenu_icon_animation.json