831402 - Tablets should use Android Spinners

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Theme and Visual Design, defect)

Description

[Sriram Ramasubramanian [:sriram]]

Tablets should use Android styled spinners in the Tabs UI to switch between different sections.

Comment 1

[Sriram Ramasubramanian [:sriram]]

Attachment: Part 1: Spinners on tablets

This patch replaces custom popup menu with standard android spinners. A bunch of styling is added to the themes.xml and styles.xml -- because that's how android takes in those values (so bad Android! so bad!).

Comment 2

[Sriram Ramasubramanian [:sriram]]

Attachment: Part 2: Split action-bar

Tablets now have a split action-bar to show add tab and extra menu.

Note: Split action-bars are used in android only for phones. This is to ensure to better use space for more common actions -- that needn't be burried under menu. They don't use it on tablets. We don't use on phones, but tablets.

Comment 3

[Sriram Ramasubramanian [:sriram]]

Attachment: Part 3: Popup above anchor

This shows the custom menu above the anchor -- in tabs menu.
Basically we need to know the height of the content view to decide if it should show up or down. To do that, we call measure when we add content.

Comment 4

[Sriram Ramasubramanian [:sriram]]

Attachment: Part 4: Action Bar dividers

This shifts the action-bar divider to be contained inside the action-bar height. We have to add an extra view in our case for 2 reasons:
1. While supporting personas, we need to show this divider. If not, the layout seems messy without it. Hence we need a solid line.
2. It could be a part of the parent container. However, in phones with tabs, the tab's selected state is cut off. Usually android applications have the entire action-bar divider color to be same as tab-selected color. Thereby, when they are applied as background, and tabs on top, they don't face any issue. We use a orange for selection and grey as divider and hence have to use a separate view for the divider.

Comment 5

[Sriram Ramasubramanian [:sriram]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/ce4d3552704d
https://hg.mozilla.org/integration/mozilla-inbound/rev/154e9ce25e41
https://hg.mozilla.org/integration/mozilla-inbound/rev/7b6b22881aaa
https://hg.mozilla.org/integration/mozilla-inbound/rev/6b7cf7305ae6

Comment 6

[Sriram Ramasubramanian [:sriram]]

Comment on attachment 702938 [details] [diff] [review]
Part 1: Spinners on tablets

[Approval Request Comment]
Bug caused by (feature/regressing bug #): New feature.
User impact if declined: Holo themed spinners will be missing in Tablets.
Testing completed (on m-c, etc.): Landed in m-c on 01/17
Risk to taking this patch (and alternatives if risky): Very low. This replaces a Button with a Spinner - which shows a DropDown by itself.
String or UUID changes made by this patch: None.

Comment 7

[Sriram Ramasubramanian [:sriram]]

Comment on attachment 702939 [details] [diff] [review]
Part 2: Split action-bar

[Approval Request Comment]
Bug caused by (feature/regressing bug #): New feature.
User impact if declined: Holo themed split action-bar on tablets.
Testing completed (on m-c, etc.): Landed in m-c on 01/17
Risk to taking this patch (and alternatives if risky): Very low. The messy "add new tab" button is moved to the bottom.
String or UUID changes made by this patch: None.

Comment 8

[Sriram Ramasubramanian [:sriram]]

Comment on attachment 702941 [details] [diff] [review]
Part 3: Popup above anchor

[Approval Request Comment]
Bug caused by (feature/regressing bug #): New feature.
User impact if declined: PopupMenu won't show.
Testing completed (on m-c, etc.): Landed in m-c on 01/17
Risk to taking this patch (and alternatives if risky): Very low. This shows the PopupMenu above the anchor. Needed for menu button at the bottom.
String or UUID changes made by this patch: None.

Comment 9

[Sriram Ramasubramanian [:sriram]]

Comment on attachment 703029 [details] [diff] [review]
Part 4: Action Bar dividers

[Approval Request Comment]
Bug caused by (feature/regressing bug #): New feature.
User impact if declined: Barely noticeable by anyone. ;)
Testing completed (on m-c, etc.): Landed in m-c on 01/17
Risk to taking this patch (and alternatives if risky): Very low. A slight change in moving the action-bar dividers up by 2dp.
String or UUID changes made by this patch: None.

Comment 10

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

https://hg.mozilla.org/mozilla-central/rev/ce4d3552704d
https://hg.mozilla.org/mozilla-central/rev/154e9ce25e41
https://hg.mozilla.org/mozilla-central/rev/7b6b22881aaa
https://hg.mozilla.org/mozilla-central/rev/6b7cf7305ae6

Comment 11

[Sriram Ramasubramanian [:sriram]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/e936ec3bd432
https://hg.mozilla.org/releases/mozilla-aurora/rev/59fd9215148a
https://hg.mozilla.org/releases/mozilla-aurora/rev/546526e62bd3
https://hg.mozilla.org/releases/mozilla-aurora/rev/c010556f8fa2