659165 - Android Download notifications are not consistent with Honeycomb theme

Status: RESOLVED DUPLICATE of bug 669636

(Firefox for Android Graveyard :: General, defect)

Description

[Tony Chung [:tchung]]

Attachment: download notification screenshot

The download notification popups on honeycomb (Android 3.1) are not consistent with the rest of the theming of the notifcation. 

See screenshot

Repro
1) install nightly android: Mozilla/5.0 (Android; Linux armv71; rv:6.0a1) Gecko/20110523 Firefox/6.0a1 Fennec/6.0a1
2) install on a Android Tablet with Honeycomb (Moto Xoom, Android 3.1)
3) launch fennec, and find a file to download (eg. Ubuntu .iso file)
4) in bottom right corner, launch the notification popups for downloads
5) Verify the text is black (need to look really closely), and the download icon is small and shoved to the left.   

Expected:
- the downloaded notification view should have white text, with a sizable icon matching the other notification layouts

Actual:
- see screenshot

Comment 1

[Wesley Johnston (:wesj)]

These text colors are hardcoded in:

http://mxr.mozilla.org/mozilla-central/source/embedding/android/resources/layout/notification_progress.xml#29

Comment 2

[Tony Chung [:tchung]]

Attachment: Phone download notification screenshot

not on the screenshot, but the completion % (on the left of progress bar is also Black color'd. 

Just including a screenshot on a Nexus S for comparison sake.

Comment 3

[Sriram Ramasubramanian [:sriram]]

I believe, this needs another notification_progress.xml in a different folder.

http://developer.android.com/guide/practices/screens_support.html#qualifiers

"res/layout/xlarge" if I'm right.
This needs to be added as a part of apk - so that android takes this new "notification_progress.xml" for honeycomb / large screens alone.

The ID can remain the same - hence this wouldn't impact the code part.

Comment 4

[Sriram Ramasubramanian [:sriram]]

"Place the new layout in its own directory, named "layout-v9", and Android will know to look for it there when the device is running API level 9 or above."

https://support.urbanairship.com/customer/portal/articles/77722-honeycomb-compatibility-for-custom-notification-layouts

Comment 5

[Sriram Ramasubramanian [:sriram]]

This has been fixed now by: https://bugzilla.mozilla.org/show_bug.cgi?id=669636