nioc / xmpp-web

Lightweight web chat client for XMPP server
GNU Affero General Public License v3.0
144 stars 20 forks source link

Provide a way to insert newlines when hasSendingEnterKey is true #70

Closed Ppjet6 closed 1 year ago

Ppjet6 commented 1 year ago

Disclaimer: I am not a user of xmpp-web myself. We've deployed it on joinjabber.org from the start and it's been of great help to our users. Thanks for developing and maintaining the project!

Is your feature request related to a problem? Please describe. In many of the clients I use, Enter is used to send. I felt users would be more used to this behaviour and so I set hasSendingEnterKey to true. But I realized there is no way to insert newlines anymore as it seems Ctrl+Enter is always used to send.

Describe the solution you'd like One way to insert newlines when Enter is used to send.

Describe alternatives you've considered

Shift+Enter seems to be used in other clients such as ConverseJS, Dino, etc.

nioc commented 1 year ago

This point of view has been discussed for many times 😉 But with time I consider Shift+Enter a good way of adding new line.

nioc commented 1 year ago

Can you confirm that Shift+Enter does not add new line? It was the expected behavior 😞

See https://github.com/nioc/xmpp-web/issues/54#issuecomment-1141390881

  • true: Control+Enter and Enter send message, Shift+Enter add new line
  • false: Control+Enter send message, Enter and Shift+Enter add new line
nioc commented 1 year ago

I just checked, it works as expected for me (Firefox 109). I close the issue, feel free to reopen it if yours users encounter issue with Shift+Enter for new line.

Ppjet6 commented 1 year ago

I was persuaded there was no way to do so. Sorry for the bother! Shift+Enter works indeed.