Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Added "Save my downloads here" setting #2110

Closed
bradleyrichter opened this issue Jun 7, 2016 · 4 comments
Closed

Added "Save my downloads here" setting #2110

bradleyrichter opened this issue Jun 7, 2016 · 4 comments

Comments

@bradleyrichter
Copy link
Contributor

bradleyrichter commented Jun 7, 2016

Test plan

#7023 (comment)


Functionality:
Providing a user-definable default downloads destination allows for a more efficient user work flow when downloading files. The default path is their downloads folder. This path can be changed by clicking on the pencil (edit) button. This opens a download location chooser dialog which is OS specific.

A toggle switch (off by default) allows the user to change the setting to ask them for the desired location upon each file download.

item label:
"Save my downloads here"

toggle switch label:
"Always ask me where to save files"

Download location sheet-dialog in Mac:
image

Spec image: (this issue in blue)
image

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Feb 3, 2017

It's really annoying not having this and selecting folder for every single download. Will add this 😃

@NejcZdovc NejcZdovc self-assigned this Feb 3, 2017
@bradleyrichter
Copy link
Contributor Author

bradleyrichter commented Feb 3, 2017 via email

NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Feb 3, 2017
Resolves brave#2110

Auditors: @bradleyrichter @bsclifton

Test Plan:
- go to preferences, general tab
- define your default download path
- toggle show dialog
- based on your settings, download dialog should be opened or not
- if you set don't show dialog, filse should be downloaded to your default set path
@NejcZdovc NejcZdovc modified the milestones: 0.13.3, 1.0.0 Feb 3, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Feb 3, 2017
Resolves brave#2110

Auditors: @bradleyrichter @bsclifton

Test Plan:
- go to preferences, general tab
- define your default download path
- toggle show dialog
- based on your settings, download dialog should be opened or not
- if you set don't show dialog, filse should be downloaded to your default set path
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Feb 12, 2017
Resolves brave#2110

Auditors: @bradleyrichter @bsclifton

Test Plan:
- go to preferences, general tab
- define your default download path
- toggle show dialog
- based on your settings, download dialog should be opened or not
- if you set don't show dialog, filse should be downloaded to your default set path
@bbondy bbondy modified the milestones: 0.13.6, 0.13.5 Feb 16, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Feb 27, 2017
Resolves brave#2110

Auditors: @bradleyrichter @bsclifton

Test Plan:
- go to preferences, general tab
- define your default download path
- toggle show dialog
- based on your settings, download dialog should be opened or not
- if you set don't show dialog, filse should be downloaded to your default set path
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Mar 3, 2017
Resolves brave#2110

Auditors: @bradleyrichter @bsclifton

Test Plan:
- go to preferences, general tab
- define your default download path
- toggle show dialog
- based on your settings, download dialog should be opened or not
- if you set don't show dialog, filse should be downloaded to your default set path
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Mar 3, 2017
Resolves brave#2110

Auditors: @bradleyrichter @bsclifton

Test Plan:
- go to preferences, general tab
- define your default download path
- toggle show dialog
- based on your settings, download dialog should be opened or not
- if you set don't show dialog, filse should be downloaded to your default set path
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Mar 8, 2017
Resolves brave#2110

Auditors: @bradleyrichter @bsclifton

Test Plan:
- go to preferences, general tab
- define your default download path
- toggle show dialog
- based on your settings, download dialog should be opened or not
- if you set don't show dialog, filse should be downloaded to your default set path

 # Added hover effect
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Mar 10, 2017
Resolves brave#2110

Auditors: @bradleyrichter @bsclifton

Test Plan:
- go to preferences, general tab
- define your default download path
- toggle show dialog
- based on your settings, download dialog should be opened or not
- if you set don't show dialog, filse should be downloaded to your default set path

 # Added hover effect
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Mar 13, 2017
Resolves brave#2110

Auditors: @bradleyrichter @bsclifton

Test Plan:
- go to preferences, general tab
- define your default download path
- toggle show dialog
- based on your settings, download dialog should be opened or not
- if you set don't show dialog, filse should be downloaded to your default set path

 # Added hover effect
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Mar 13, 2017
Resolves brave#2110

Auditors: @bradleyrichter @bsclifton

Test Plan:
- go to preferences, general tab
- define your default download path
- toggle show dialog
- based on your settings, download dialog should be opened or not
- if you set don't show dialog, filse should be downloaded to your default set path

 # Added hover effect
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Mar 13, 2017
Resolves brave#2110

Auditors: @bradleyrichter @bsclifton

Test Plan:
- go to preferences, general tab
- define your default download path
- toggle show dialog
- based on your settings, download dialog should be opened or not
- if you set don't show dialog, filse should be downloaded to your default set path

 # Added hover effect
@alexwykoff alexwykoff mentioned this issue Mar 14, 2017
44 tasks
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Mar 16, 2017
Resolves brave#2110

Auditors: @bradleyrichter @bsclifton

Test Plan:
- go to preferences, general tab
- define your default download path
- toggle show dialog
- based on your settings, download dialog should be opened or not
- if you set don't show dialog, filse should be downloaded to your default set path

 # Added hover effect
bsclifton added a commit that referenced this issue Mar 16, 2017
@cndouglas
Copy link

A toggle switch (off by default) allows the user to change the setting to ask them for the desired location upon each file download.

This is what I get on a fresh install:
image
Should the switch be on or off by default?

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Mar 24, 2017

I set it to on by default yes. If we go with the off by default, then I think we would need to set default path when browser is first lunched (or when you go to preferences, we use same function for path determination that we use it on the save event), otherwise it would be strange to have empty input and option to off

cc @bradleyrichter

@alexwykoff alexwykoff changed the title General Settings: "Save my downloads here" setting Added "Save my downloads here" setting Mar 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.