New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add option to delete files when clearing finished downloads #2717
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your improvement. I am sorry for the late reply. There are two small things which I'd like to note.
@kapodamy is the expert for everything regarding the downloader. Do you have any thoughts on this?
app/src/main/java/us/shandian/giga/ui/adapter/MissionAdapter.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. There are a few small things left.
@kapodamy Do you have any thoughts on this PR? You are the master of the downloader :)
app/src/main/java/us/shandian/giga/ui/adapter/MissionAdapter.java
Outdated
Show resolved
Hide resolved
app/src/main/res/values/strings.xml
Outdated
@@ -587,5 +587,6 @@ | |||
<string name="downloads_storage_use_saf_title">Use SAF</string> | |||
<string name="downloads_storage_use_saf_summary">The Storage Access Framework allows downloads to an external SD card.\nNote: some devices are not compatible</string> | |||
<string name="choose_instance_prompt">Choose an instance</string> | |||
|
|||
<string name="delete_downloaded_files">Delete downloaded files</string> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please move these strings to up the other download related strings (line 573).
app/src/main/java/us/shandian/giga/ui/adapter/MissionAdapter.java
Outdated
Show resolved
Hide resolved
Thanks again! |
This PR fixes issue #2267 by adding a checkbox in alert dialog that allows user to decide whether to delete downloaded files when clearing finished downloads.