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

"Return" sends message immediately without warning #75

Closed
Katermerlin opened this issue Jul 24, 2020 · 4 comments · Fixed by #93
Closed

"Return" sends message immediately without warning #75

Katermerlin opened this issue Jul 24, 2020 · 4 comments · Fixed by #93
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@Katermerlin
Copy link

There should be a switch for prevention of sending if the "Return" key is hit, like in the Android version of Whatsapp. It happens often that Return is hit by mistake and the message is sent. The Return should work only for vertical spacing, sending only by activating the "Send" - arrow...

@amanharwara amanharwara added the enhancement New feature or request label Jul 26, 2020
@amanharwara amanharwara added the help wanted Extra attention is needed label Aug 13, 2020
@amanharwara
Copy link
Owner

So far from what I've tried to implement this, none of it seems to work. This is mainly because of how the message box seems to work.

I'm not sure if I can implement this but PRs are welcome.

@Katermerlin
Copy link
Author

Katermerlin commented Sep 25, 2020

I'm not sure what "PR" is...
Nevertheless, IMHO it should be possible to "catch" the Enter-command before it is sent to the system... I think the "Enter" is the same as the CRLF, isnt'it?
It's because I was told that any command sent by the keyboard can be catched before execution resp. execution can be prevented... Maybe this hint helps...

There should be a simple return to the next line instead af "enter"...

@amanharwara
Copy link
Owner

Will release a new version with this feature in a day or so.

@amanharwara
Copy link
Owner

@Katermerlin this feature has now been implemented in v3.16.0. Please try it out. You will need to turn it on in the settings first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants