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

Remove Attachment - Feature request #386

Open
km4ack opened this issue Dec 25, 2022 · 4 comments
Open

Remove Attachment - Feature request #386

km4ack opened this issue Dec 25, 2022 · 4 comments

Comments

@km4ack
Copy link

km4ack commented Dec 25, 2022

I don't see a way to remove an attachment after it has been added to an outgoing email. Could this be added?

@xylo04
Copy link
Contributor

xylo04 commented Dec 27, 2022

Why would you ever want to do that? 😆

Yes, this seems very reasonable and doable.

@km4ack
Copy link
Author

km4ack commented Dec 27, 2022

Thanks @xylo04

@xylo04
Copy link
Contributor

xylo04 commented Dec 28, 2022

While working on this feature request, I also came across a related bug that Pat doesn't seem to actually attach more than one attachment to a message at a time. The compose dialog shows multiple attachments, but only the last one uploaded gets attached when posted to the outbox. So that's fun.

Screenshot from 2022-12-27 19-36-57

Screenshot from 2022-12-27 19-37-36


Edit for clarification: you can attach multiple files by selecting multiple in the file system dialog, but you can't attach one file at a time. Only the files in the last file system selection are actually part of the HTML input element.

@xylo04
Copy link
Contributor

xylo04 commented Dec 30, 2022

I've started work in this branch but need to come up with a way to dynamically add input elements so we can add attachment files more than once.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants