810048 - Show filename in notifcation Title during download

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Download Manager, defect)

Description

[Wesley Johnston (:wesj)]

We current show a notification with a title of "Downloads" and a message with the filename, but when we're showing progress, the message isn't shown.

We should show the filename in the notication's Title (and show some state in the message "Starting", "Done", "Not enough space", "Failed", etc.)

Comment 1

[Wesley Johnston (:wesj)]

Attachment: Patch

This changes our notifications to just have the filename as the title, and status as the msg. i.e. If I was downloading file.txt I would see:

|--| file.txt
|--| Download starting

|--| file.txt
|--| 50% ==============

|--| file.txt
|--| Download complete

I removed the prefix "Downloading: " from the title because it felt like wasted space on a small screen, but I'm open to talking about it. I'll mark this as feedback needed from ibarlow as well.

Comment 2

[:Margaret Leibovic]

Comment on attachment 679893 [details] [diff] [review]
Patch

Review of attachment 679893 [details] [diff] [review]:
-----------------------------------------------------------------

Shorter titles sound good to me. Just one comment to address before landing.

::: mobile/android/locales/en-US/chrome/browser.properties
@@ -12,5 @@
>  alertAddonsInstalled=Installation complete. Restart required.
>  alertAddonsInstalledNoRestart=Installation complete
>  alertAddonsFail=Installation failed
>  
> -alertDownloads=Downloads

I thought you accidentally deleted this, but now I see that it's not going to be used anymore because we won't call showAlert() without a title. If that's the case, let's get rid of that check in showAlert() to provide a default title:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/downloads.js#83

Comment 3

[Kevin Brosnan [Ex-Mozilla]]

How is this different than bug 741654.

Comment 4

[Wesley Johnston (:wesj)]

This has to do with the native android notifications. That has to do with about:downloads.

Comment 5

[Ian Barlow (:ibarlow)]

(In reply to Wesley Johnston (:wesj) from comment #1)
> Created attachment 679893 [details] [diff] [review]
> Patch
> 
> This changes our notifications to just have the filename as the title, and
> status as the msg. i.e. If I was downloading file.txt I would see:
> 
> |--| file.txt
> |--| Download starting
> 
> |--| file.txt
> |--| 50% ==============
> 
> |--| file.txt
> |--| Download complete
> 
> I removed the prefix "Downloading: " from the title because it felt like
> wasted space on a small screen, but I'm open to talking about it. I'll mark
> this as feedback needed from ibarlow as well.

This looks fine to me. In addition to saving space for a file name, showing download progress makes the word "Downloading" in the title bar somewhat redundant anyway.

Comment 6

[Wesley Johnston (:wesj)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b0384bf925aa

Comment 7

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/b0384bf925aa

Comment 8

[Cristian Nicolae (:xti)]

The filename is displayed in Notification Bar as expected when downloads starts, when it's in progress and when it's done. Closing bug as verified fixed on:

Firefox 19.0a1 (2012-11-16)
Device: Galaxy S2
OS: Android 4.0.3