Skip to content
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

downloads_dir cannot create a directory, and files are downloaded before user confirmation #158

Closed
Kabouik opened this issue Nov 18, 2023 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@Kabouik
Copy link

Kabouik commented Nov 18, 2023

Description:
The downloads_dir option cannot create a directory, and will default to ~/Downloads if the set path does not exist. This can be an issue if the user wants to download to a /tmp subfolder for instance. It gracefully falls back to ~/Downloads, but does not show a warning or create the missing directory, which I think would be best.

Also, when pressing ^R to download a file, there is a prompt indicating the path and asking to cancel or confirm, but files are downloaded even when cancelling. This is the case at least for attachments sent with attachment_send_type=0, but perhaps other attachments too.

Environment:

  • Version: 3.97
  • OS / distro: Droidian
@Kabouik Kabouik added the bug Something isn't working label Nov 18, 2023
@d99kris d99kris closed this as completed in 7f721ec Dec 9, 2023
@d99kris
Copy link
Owner

d99kris commented Dec 9, 2023

Hi @Kabouik - I've added support for creating downloads_dir if specified directory does not exist.

As for the the downloading of files, the dialog is intended to just be a notification message (so the user knows where the file ended up, and with what name). Thus, I've removed showing the "cancel" key binding in the help bar when showing this notification.

@Kabouik
Copy link
Author

Kabouik commented Dec 9, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants