Closed Bug 862996 Opened 8 years ago Closed 8 years ago

Put text headings back in the tab menu

Categories

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

ARM
Android
defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 23
Tracking Status
firefox23 --- verified

People

(Reporter: ibarlow, Assigned: sriram)

References

Details

(Whiteboard: ui-hackathon)

Attachments

(1 file)

http://cl.ly/image/0d3q243A2Q3A

We've been getting a lot of negative feedback that the icons are unclear on their own, and using text labels instead is just a nicer, more communicative design. The fact that someone pointed out that they looked like a moustache and a tophat didn't help, either.


Let's put text back in the tabs everywhere except in the tablet sidebar, where we'll keep the icons due to space constraints.

I realize there are potential localization concerns, but given the smaller text label size and fewer elements in the title bar than before, we have more room to work with now. 


For the text, please use Roboto Bold 12sp, white with 60% opacity
Blocks: 817675
(In reply to Ian Barlow (:ibarlow) from comment #0)
> The fact that someone pointed out that they looked like a moustache
> and a tophat didn't help, either.

http://www.am.ub.edu/~carrasco/bichos/porco2.jpg
I rest my case. :)
Attached patch PatchSplinter Review
OMG! OMG! OMG!!!
So, I made the ImageButton a Button. Since Button inherits from TextView, the "iconified" one is just a compound-drawable. If it displays the text, it displays it happily! :O :O
We are becoming super efficient in using very less layouts! :O
Attachment #738780 - Flags: review?(mark.finkle)
Attachment #738780 - Flags: review?(mark.finkle) → review+
Whiteboard: ui-hackathon
Land it?
OS: Mac OS X → Android
Hardware: x86 → ARM
Status: NEW → ASSIGNED
Do it do it
Version: unspecified → Trunk
https://hg.mozilla.org/mozilla-central/rev/14be1f919f01
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 23
Status: RESOLVED → VERIFIED
Depends on: 867566
Depends on: 874014
No longer depends on: 867566
You need to log in before you can comment on or make changes to this bug.