Closed Bug 702264 Opened 12 years ago Closed 12 years ago

[ICS] - ActionBar above navigation bar


(Firefox for Android Graveyard :: General, defect, P4)



(Not tracked)



(Reporter: aaronmt, Assigned: sriram)


(Whiteboard: [ICS])


(3 files)

Attached image ICS - Nightly (11/14)
See screenshot.

Samsung Nexus S (Android 4.0)
Priority: -- → P4
Summary: [ICS] - Progress bar above navigation bar → [ICS] - ActionBar above navigation bar
I'll have a look into this.
Assignee: nobody →
Assignee: → sriram
Also seen on Android 3.2.
This patch does the required cleanup in AndroidManifest to use common theme names (based off "Gecko" shadowing android themes) between Honeycomb+ and Gingerbread-.
This way of using themes ensure that the AndroidManifest remains the same, while the themes change based on which OS version it is running.
Also, this added a new styles.xml to use honeycomb specific styles. With this, the list in AwesomeBar uses native styling.

AndroidManifest.xml was also cleared to just use the requires flags. Few activities had "MAIN" intent-filter with them -- which wasn't needed. Also, the activities like GeckoPreferences doesn't check for the "intent-filter" in them. Hence they have been removed.
Attachment #576551 - Flags: review?(mark.finkle)
This patch adds "ActionBar" support to Honeycomb and ICS. They are basically handled in themes as much as possible. Using "displayOptions" with themes will be more elegant. However, they will fragment the themes so much for each activity. Hence they are handled during onCreate.

I would be happy to revisit cleaning that part later if possible. I couldn't find any documentation that supports using "displayOptions" directly with the activity.
Attachment #576553 - Flags: review?(mark.finkle)
Attachment #576553 - Flags: review?(mark.finkle) → review+
Attachment #576551 - Flags: review?(mark.finkle) → review+
Verified on Samsung Galaxy Nexus (Android 4.0.1).
Nightly 12.0a1 (2012-01-16)
Aurora 11.0a2 (2012-01-17)
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.