Let's update the icons to go better with the new grey header.
OS: Mac OS X → Android
Hardware: x86 → All
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 :)
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.