Implement doorhanger sub-menu animation and new icon

NEW
Unassigned

Status

()

Firefox
Menus
P4
normal
a month ago
5 days ago

People

(Reporter: amylee, Unassigned)

Tracking

58 Branch
Points:
---

Firefox Tracking Flags

(firefox58 affected)

Details

Attachments

(4 attachments)

(Reporter)

Description

a month ago
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.
(Reporter)

Comment 1

a month ago
Created attachment 8918355 [details]
Options.svg
(Reporter)

Comment 2

a month ago
Created attachment 8918356 [details]
Arrow.svg
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?
Flags: needinfo?(amlee)
(Reporter)

Comment 4

a month ago
(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.
Flags: needinfo?(amlee) → needinfo?(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.
Flags: needinfo?(shorlander)
Amy, would you be able to supply the SVG sprite?
Flags: needinfo?(amlee)
(Reporter)

Comment 7

a month ago
Created attachment 8920681 [details]
submenu_icon_animation.svg

Here's the sprite
Flags: needinfo?(amlee)
(Reporter)

Comment 8

a month ago
Created attachment 8920682 [details]
submenu_icon_animation.json
Priority: -- → P4
You need to log in before you can comment on or make changes to this bug.