Skip to content

fix: attachments opening handling#371

Merged
boxdot merged 1 commit intomainfrom
fix-attachments-handling
Feb 20, 2025
Merged

fix: attachments opening handling#371
boxdot merged 1 commit intomainfrom
fix-attachments-handling

Conversation

@boxdot
Copy link
Copy Markdown
Owner

@boxdot boxdot commented Feb 18, 2025

Remove invalid character from attachment file names. This allows to open the attachment directly from terminal by clicking on it.

Also try to open url or attachment on enter when selected a message and not having any input.

Remove invalid character from attachment file names. This allows to open
the attachment directly from terminal by clicking on it.

Also try to open url or attachment on enter when selected a message and
not having any input.
@hrdl-github
Copy link
Copy Markdown
Contributor

In my own tree I usually comment out the two instances of reset_message_selection(), as I usually end up interacting with more messages (e.g. opening more files) or want to copy the URL afterwards. Would you prefer making this behaviour configurable?

@boxdot
Copy link
Copy Markdown
Owner Author

boxdot commented Feb 18, 2025

This sounds like a good default behavior. What might be missing, is visual feedback that pressing Enter actually did something. Maybe we can add a small pop up (could be also reused for error messages).

EDIT: I just tried it out. With urls it is a little bit weird, because it feels like nothing happened. Maybe a configuration parameter makes more sense.

@boxdot boxdot merged commit e92d191 into main Feb 20, 2025
16 checks passed
@boxdot boxdot deleted the fix-attachments-handling branch February 20, 2025 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants