[go: up one dir, main page]

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

[Bug] Clicking on an attachment in draft removed it #398

Open
lostmsu opened this issue Sep 26, 2024 · 2 comments
Open

[Bug] Clicking on an attachment in draft removed it #398

lostmsu opened this issue Sep 26, 2024 · 2 comments
Assignees
Labels
bug Something isn't working UI Issues related to design or styling.

Comments

@lostmsu
Copy link
lostmsu commented Sep 26, 2024

Describe the bug
I clicked an attachment in draft, and it just got removed.

Expected behavior
The attachment should get opened.

Screenshots
image

Me just before clicking at that exact spot.

Additional context
Main branch from yesterday, commit 6782836

@lostmsu lostmsu added the bug Something isn't working label Sep 26, 2024
@bkaankose
Copy link
Owner

This didn't make it to the recent build, but I'd like to improve this part as well.

When you attach a file to composer, it's loaded into memory. You can as well delete the physical file from the disk and Wino can still send the attachment. Therefore opening the file for preview requires content to be saved to temporary folder in case of the file is deleted.

I'll have it implemented for the next version.

@bkaankose bkaankose added the UI Issues related to design or styling. label Sep 30, 2024
@lostmsu
Copy link
Author
lostmsu commented Sep 30, 2024

There's also the issue that this particular area of the "button" activated the delete action instead of the open action.

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

No branches or pull requests

2 participants