Closed Bug 766654 Opened 12 years ago Closed 12 years ago

Pasting a URL into download manager to start a download does not work with the new download panel

Categories

(Firefox :: Downloads Panel, defect)

16 Branch
defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 19

People

(Reporter: mbrubeck, Assigned: mconley)

References

Details

(Keywords: regression, ux-efficiency)

Attachments

(1 file, 2 obsolete files)

In the old download manager, pasting a URL causes Firefox to download a URL. This shortcut is useful because it's the one of the only ways to make Firefox start a download immediately rather than loading and viewing the URL. This is no longer possible in the new downloads panel UI.
loss of functionality
Assignee: nobody → mconley
Attached patch WIP Patch 1 (obsolete) (deleted) — Splinter Review
Checkpointing work here for the night. I still have a few focus issues to work out, but this basically works.
Attached patch Patch v1 (obsolete) (deleted) — Splinter Review
Here's my first run at this.
Attachment #679520 - Attachment is obsolete: true
Attachment #680158 - Flags: review?(mak77)
Comment on attachment 680158 [details] [diff] [review] Patch v1 Review of attachment 680158 [details] [diff] [review]: ----------------------------------------------------------------- ::: browser/components/downloads/content/downloads.js @@ +308,5 @@ > +#else > + aEvent.ctrlKey; > +#endif > + > + if (pasting) { invert this and early return, so less indentation. @@ +330,5 @@ > + if (!url) { > + return; > + } > + > + let uri = Services.io.newURI(url, null, null); nit: lots of empty space above, maybe compact code a bit
Attachment #680158 - Flags: review?(mak77) → review+
Attached patch Patch v2 (r+'d by mak) (deleted) — Splinter Review
Thanks for the r+! Requested changes made.
Attachment #680158 - Attachment is obsolete: true
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 19
Verified as fixed on the latest Nightly - pasting a URL in the downloads panel starts the download. Verified on Ubuntu 12.10, Mac OS X 10.7 and Windows 7: Mozilla/5.0 (X11; Linux i686; rv:20.0) Gecko/20121203 Firefox/20.0 Build ID: 20121203030801 Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:20.0) Gecko/20121203 Firefox/20.0 Build ID: 20121203030801 Mozilla/5.0 (Windows NT 6.1; rv:20.0) Gecko/20121202 Firefox/20.0 Build ID: 20121202030723
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: