Closed Bug 817749 Opened 12 years ago Closed 12 years ago

Phone - adjust tab title bar to help differentiate main menu from tab menu

Categories

(Firefox for Android Graveyard :: General, defect)

x86
macOS
defect
Not set
normal

Tracking

(firefox20 verified, firefox21 verified)

VERIFIED FIXED
Firefox 20
Tracking Status
firefox20 --- verified
firefox21 --- verified

People

(Reporter: ibarlow, Assigned: sriram)

References

Details

Attachments

(4 files)

Attached image mockup (deleted) —
Since our new tab tray designs require the use of an action bar menu, we should tweak the title bar design to make it easier to visually distinguish between the main title bar menu icon, and the tab title bar menu icon. We can probably do this with a simple transition that would involve 1. animating the main menu icon off the screen when the tab tray is opened 2. extending the grey tab bar background all the way across the header 3. show a tab menu icon in a slightly different colour.
Is the menu button's background black or grey? There will be an abrupt switch if its grey on tabs-menu.
That's why we need a quick transition from one title bar to the other.
Attached patch Part 1: Flatten the toolbar (deleted) — Splinter Review
This flattens the tabs-tray toolbar. No more curvy buttons in tabs-panel. Yaay! no more personas for them!! yaay!! Less XML files! Yaay!! But we need new blue menu button similar to "+" button. That's the next patch.
Attachment #694008 - Flags: review?(mark.finkle)
Attached patch Part 2: No sharing (deleted) — Splinter Review
We have a solid square button. We don't need a delegate anymore.
Attachment #694060 - Flags: review?(mark.finkle)
Attached patch Part 3: Menu icons in blue (deleted) — Splinter Review
And the new menu icons to look good on the tabs-panel.
Attachment #694061 - Flags: review?(mark.finkle)
Attachment #694008 - Flags: review?(mark.finkle) → review+
Attachment #694060 - Flags: review?(mark.finkle) → review+
Comment on attachment 694061 [details] [diff] [review] Part 3: Menu icons in blue nice set of patches
Attachment #694061 - Flags: review?(mark.finkle) → review+
We still need a transition, as Ian indicates. That could be a followup bug.
Assignee: nobody → sriram
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 20
Depends on: 824276
The changes were applied on both Nightly and Aurora. Closing bug as verified fixed on: Firefox for Android Version: 21.0a1 (2013-01-29) Device: Galaxy R OS: Android 2.3.4
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: