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

Send chat message on enter #463

Closed
jancborchardt opened this issue Nov 3, 2017 · 5 comments
Closed

Send chat message on enter #463

jancborchardt opened this issue Nov 3, 2017 · 5 comments

Comments

@jancborchardt
Copy link
Member

Currently enter adds a line break. That’s quite annoying because usually messages are short, and most platforms submit on enter.

We could do Ctrl-Enter to insert line breaks if necessary.

cc @danxuliu

@jancborchardt jancborchardt added this to the 3.0 (Nextcloud 13) milestone Nov 3, 2017
@danxuliu
Copy link
Member

danxuliu commented Nov 3, 2017

That’s quite annoying because usually messages are short, and most platforms submit on enter.

Agreed.

It is simply a matter of changing the condition in _onTypeComment. I could push a new commit to #462, but I would prefer to fix this in a different pull request once that one is merged to better keep track of the issue (as from what I have seen Ctrl+Enter does not create a new line by default on my system, but Shift+Enter does, and right now I am not sure if it is a general behaviour across operating systems/browsers or it depends on each one).

@jancborchardt
Copy link
Member Author

Right, ok. Ideally we should do the same for comments in the Files sidebar too so it's consistent.

@danxuliu
Copy link
Member

@jancborchardt

Ideally we should do the same for comments in the Files sidebar too so it's consistent.

I can import the change in the Comments app if you want (although comments may be more prone to longer, more elaborated texts than chat messages and thus the current approach of new line on enter is better, I do not know).

@jancborchardt
Copy link
Member Author

@danxuliu yes, porting it would be great! The issue atm is more that not really a lot of people know we have Comment functionality, so we need to make it as dead-simple to use as possible. :)

@danxuliu
Copy link
Member

@jancborchardt

porting it would be great!

Here you have: nextcloud/server#7252 ;-)

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

3 participants