Matt Brubeck (:mbrubeck) Assignee
	Description • 11 years ago

shorlander requested (in person at the summit) that the download button in the navbar should be visible only when there is a download in progress.

The first patch on this bug just moves some code around to separate generic "circular progress" styles from specific download button styles.

	Matt Brubeck (:mbrubeck) Assignee
	Comment 1 • 11 years ago

This hides the download button when there is no download in progress.

Note: Since the download button is no longer in the navbar at "dl-request" time, it no longer makes sense to show the navbar then; instead we can show it when the download actually starts and the button appears.

	Matt Brubeck (:mbrubeck) Assignee
	Comment 2 • 11 years ago

Also per shorlander's request, when I demoed parts 1 and 2 to him.  I know I resisted this previously, but trying it both ways, I agree that this improves the UX.

With this patch, we don't hide the download button until the user has chosen an action from the download-complete infobar.

	Marina Samuel [:emtwo]
	Comment 3 • 11 years ago

Comment on attachment 813875 [details] [diff] [review]
part 2: show download button only during download progress

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

Looks good!

This patch reminded me that someone at the summit while demo-ing mentioned that it's odd that the home page has some empty space on the right next to the urlbar where the toolbar buttons show up in a new tab. Do we or should we have a plan for a different UX for this?

::: browser/metro/base/content/downloads.js
@@ -92,5 @@
>            break;
>        }
>      }
>      if (this.manager.activeDownloadCount) {
> -      Services.obs.notifyObservers(null, "dl-request", "");

I believe we have another call like this in HelperAppDialog.js that we can get rid of.

	Marina Samuel [:emtwo]
	Comment 4 • 11 years ago

Comment on attachment 813921 [details] [diff] [review]
part 3: Make the download-complete notification toggleable

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

Looks awesome! I like this approach :)

	Matt Brubeck (:mbrubeck) Assignee
	Comment 5 • 11 years ago

https://hg.mozilla.org/integration/fx-team/rev/7c65c5d7ad68
https://hg.mozilla.org/integration/fx-team/rev/b417845bc606
https://hg.mozilla.org/integration/fx-team/rev/3740ef3fb8a1

	Carsten Book [:Tomcat]
	Comment 6 • 11 years ago

https://hg.mozilla.org/mozilla-central/rev/7c65c5d7ad68
https://hg.mozilla.org/mozilla-central/rev/b417845bc606
https://hg.mozilla.org/mozilla-central/rev/3740ef3fb8a1

	Cornel Ionce [:noni] [Hubs QA]
	Comment 7 • 11 years ago

Mozilla/5.0 (Windows NT 6.2; WOW64; rv:27.0) Gecko/20100101 Firefox/27.0
Mozilla/5.0 (Windows NT 6.2; rv:27.0) Gecko/20100101 Firefox/27.0

Verified on the latest nightly (Build ID: 20131016030202)

If no download is started, the downloads button is not visible.
Could not completely verify this due to bug 927729, the circle won't disappear after the download is completed.

	Cornel Ionce [:noni] [Hubs QA]
	Comment 8 • 11 years ago

Mozilla/5.0 (Windows NT 6.2; WOW64; rv:27.0) Gecko/20100101 Firefox/27.0
Mozilla/5.0 (Windows NT 6.2; rv:27.0) Gecko/20100101 Firefox/27.0

Verified as fixed on latest Nightly (build ID: 20131028030205).

When a download is completed and the user selects "run" or "show in folder", the download button is not shown anymore. Also, the button is hidden when if user will cancel the download.