794088 - Menu: update icons, add submenu indicator

Status: VERIFIED FIXED

(Firefox for Android :: General, defect)

Description

[Ian Barlow (:ibarlow)]

Now that we have a working submenu system, we just need to update the icons, and add some kind of indicator for menu items that have submenus.

Mockup: http://cl.ly/image/0c0l400D460G

Assets are attached

Comment 1

[Ian Barlow (:ibarlow)]

Hm, assets wouldn't upload, here is a link to them: http://cl.ly/3V1R2C1r0b1G

Comment 2

[Sriram Ramasubramanian [:sriram]]

Attachment: Patch

This replaces icons, fixes the layout to move things a bit.

Comment 3

[Sriram Ramasubramanian [:sriram]]

Attachment: Patch (2/2)

We earlier have a small bevel that differentiated the menu from the page. The new plain color required a top shadow. Hence replacing it for the custom menu on phones with h/w menu button.

Comment 4

[Sriram Ramasubramanian [:sriram]]

In Inbound:
https://hg.mozilla.org/integration/mozilla-inbound/rev/56814cafb2ad
https://hg.mozilla.org/integration/mozilla-inbound/rev/0378831600de
Yaay!

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/56814cafb2ad
https://hg.mozilla.org/mozilla-central/rev/0378831600de

Comment 6

[Aaron Train [:aaronmt]]

Finally :-)