[TABLETUI] Update Actionbar icons

VERIFIED FIXED in Firefox 9

Status

defect
VERIFIED FIXED
8 years ago
8 years ago

People

(Reporter: ibarlow, Assigned: wesj)

Tracking

Firefox 9
Firefox 9
All
Android
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [inbound])

Attachments

(3 attachments)

Posted file New icon assets
Let's update the icons to go better with the new grey header.
Blocks: 655762
OS: Mac OS X → Android
Hardware: x86 → All
Posted patch Patch v1Splinter Review
I accidentally combined this with a patch for bug 685289 (same thing for the honeycomb app menu), but don't see any huge reason to pick them apart. This does look a bit funny to me right now though. Maybe because of some other patches (circle back button, conditional forward, etc. that haven't landed?). Screenshot at:

http://people.mozilla.com/~wjohnston/screenshots/newicons.png

if its deemed too ugly I'll come back and see what else we need to change.
Assignee: nobody → wjohnston
Attachment #561242 - Flags: review?(mark.finkle)
Comment on attachment 561242 [details] [diff] [review]
Patch v1

looks OK to me
Attachment #561242 - Flags: review?(mark.finkle) → review+
Wesj, looks good for now! The circle back and conditional forward bugs will pull it all together :)
https://hg.mozilla.org/mozilla-central/rev/3b962e7590f1
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 9
Verified fixed on:
Mozilla/5.0 (Android;Linux armv7l;rv:9.0a1)Gecko/20110922
Firefox/9.0a1 Fennec/9.0a1
Device: Acer ICONIA A500
OS: Android 3.1
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.