Closed Bug 1710926 Opened 3 years ago Closed 2 years ago

[meta] Outreachy 2021 - Downloads panel improvements

Categories

(Firefox :: Downloads Panel, task, P3)

task

Tracking

()

RESOLVED FIXED

People

(Reporter: mtigley, Unassigned)

References

(Depends on 4 open bugs)

Details

(Keywords: meta)

Attachments

(1 file)

Meta for the "Improve the Firefox Downloads Panel" Outreachy project. This bug will be tracking that work.

All this work will be hidden behind a pref that will be introduced in Bug 1710929, so that we have the option to turn off/on the feature if needed.

Depends on: 1710929
Component: Bookmarks & History → Downloads Panel
Depends on: 1709129
Depends on: 1710933
Depends on: 1710941
Depends on: 1711049
Depends on: 453455
Depends on: 1711053

Depending on how many people are assigned to this project, an achievable subset might be limited to bug 1711049. I have provided a few pointers there.

The rest of the dependencies of this bug, at the moment, are mostly equivalent to this set of meta-bugs:

https://bugzilla.mozilla.org/showdependencytree.cgi?id=1270405&hide_resolved=1

Depends on: 1712997
Depends on: 1714107
Depends on: 1716564
Depends on: 1712219
Depends on: 1716217
Depends on: 1718782

It looks like telemetry bug is missing: https://bugzilla.mozilla.org/show_bug.cgi?id=1706355

Depends on: 1719892
Depends on: 1719895
Depends on: 1719897
Depends on: 1719900
Depends on: 1719901
Depends on: 1719904
Depends on: 1724319
Depends on: 1723712
No longer blocks: 1727075
Depends on: 1729583
Depends on: 1730891
Depends on: 1731086
Depends on: 1731110
Depends on: 1731668
Depends on: 1732347
No longer depends on: 1719904
Depends on: 1732893
Depends on: 1733492
Depends on: 1733587
Depends on: 1735414
Depends on: 1737696

Hi! I'm a user who just noticed the new download panel behaviour. In particular, I was saving a number of small files (so each completed before the next began), from a number of tabs (further defeating any heuristics that could have helped), and was rather annoyed at how it would re-open itself with every single file saved. Searching for information about this behaviour, it looks like the title of bug 1709129 muses about an opt-out, but either one was never implemented, or it's very well hidden, so my only apparent option is to choose between daily annoyance, or disabling the improvements_to_download_pane pref as a short-term workaround.

It'd be much appreciated by myself, and probably a decent handful of other users, if there were a more fine-grained way to turn off that one specific bit of functionality before improvements_to_download_pane becomes the default and reinforces the usual "they changed it, now it sucks" resentment that comes with nearly every UI change.

(In reply to Uristqwerty from comment #3)

Hi! I'm a user who just noticed the new download panel behaviour. In particular, I was saving a number of small files (so each completed before the next began), from a number of tabs (further defeating any heuristics that could have helped), and was rather annoyed at how it would re-open itself with every single file saved. Searching for information about this behaviour, it looks like the title of bug 1709129 muses about an opt-out, but either one was never implemented, or it's very well hidden, so my only apparent option is to choose between daily annoyance, or disabling the improvements_to_download_pane pref as a short-term workaround.

It'd be much appreciated by myself, and probably a decent handful of other users, if there were a more fine-grained way to turn off that one specific bit of functionality before improvements_to_download_pane becomes the default and reinforces the usual "they changed it, now it sucks" resentment that comes with nearly every UI change.

Please file a new bug blocking bug 1733587. You can use this link. See also the documentation and FAQ at https://docs.google.com/document/d/1H0Uw7oNgATAN1Ji_MYj7loaY_H-GUmnkNKvwraeHWgQ/edit .

Flags: needinfo?(Uristqwerty)

I'm finding that this new behavior is creating duplicate files that have no contents. I went looking for errors and looked at the source code that relies on this pref, but I'm really not sure what's going on. Like I expressed before in a previous comment on one of these bugs, I think the current behavior is a little frustrating so I turned it off a couple weeks ago. I guess due to some migration code it came back on automatically after I updated recently.

And I started seeing that every time I downloaded a file that was set to "Open in Nightly" in about:preferences, it would save the file twice, but the first copy would be empty and the second copy would be named filename-1.pdf or whatever. I had PDF and SVG files both set to open in firefox. And every time I downloaded one of these, instead of asking me whether I want to open in nightly, open in another handler, or save the file, it would go right to the Windows save as dialog. Upon choosing the filename and saving, it would automatically open the file in firefox and (no idea why) it would open the downloads panel too.

The behavior is kinda weird and hard to explain so check out my attachment. I have no insights on where this is coming from. Just the observation that it only affects filetypes set to "Open in Nightly." It doesn't seem to matter what kind of file type handler is set up in the Windows registry. I tried it with PDF which is set (in the registry) to open in Firefox, then tried with SVG which is set to open in VS Code, then tried deleting the SVG file handler altogether. All 3 scenarios exhibit the same issue. But any file type not set to "Open in Nightly" does not have the issue. Finally, disabling the pref browser.download.improvements_to_download_panel eliminates the issue altogether of course.

(In reply to aminomancer from comment #5)

Created attachment 9248523 [details]
File types set to "Open in Nightly" get downloaded twice for some reason

I'm finding that this new behavior is creating duplicate files that have no contents.

This is now tracked in bug 1738601.

(In reply to Uristqwerty from comment #3)

It'd be much appreciated by myself, and probably a decent handful of other users, if there were a more fine-grained way to turn off [opening the downloads panel]

This is now tracked in bug 1738372.

For future people considering commenting here: please do not do that. If you're having issues with this behaviour, file a new bug that blocks enabling this feature on release, after reviewing this doc describing the changes.

Flags: needinfo?(Uristqwerty)
Depends on: 1744297
Depends on: 1752676
Depends on: 1753096
Depends on: 1759756
No longer depends on: 1759756

The Outreachy project itself is now long finished so I'm going to close out this meta.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: