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

Enter/Return key should not post chat messages #8066

Closed
Alys opened this issue Sep 25, 2016 · 4 comments
Closed

Enter/Return key should not post chat messages #8066

Alys opened this issue Sep 25, 2016 · 4 comments

Comments

@Alys
Copy link
Contributor

Alys commented Sep 25, 2016

Currently, when typing a message in the Tavern, guilds, or party, hitting the keyboard's Enter key will post the message. This causes users who aren't familiar with this behaviour to post multiple messages when they meant to post only one.

This should be changed so that Enter does nothing but make a new line (as Shift-Enter does now).

This is important because we want to introduce an anti-spam limit on posts (#8060) and we don't want users to accidentally exceed the limit by posting messages when they don't mean to.

Users who want to use keyboard controls to post messages can do Tab-Enter or Tab-Space (Tab moves focus to the send button and Enter or Space "clicks" the button). This behaviour must keep working. We can inform the Socialites about that so that they can share it to people who aren't happy with the removal of the Enter key's function.

(We could change Shift-Enter so that it posts the message, but many users are already used to using Shift-Enter to insert line breaks, so I personally think that changing that isn't a good idea at the moment. Perhaps a few months after Enter stops posting messages we could consider making Shift-Enter post them.)

Any coder who wants to work on this is welcome to post here to claim it, but please do that only if you can start now and finish ideally in a couple of weeks. Please give a progress report after a week so your claim doesn't expire.

FYI @lemoness

@crookedneighbor
Copy link
Contributor

I think I logged an issue about this already. My suggestion was to use ctrl/cmd + enter to send, like Github. (I believe I've already implemented that bit)

@crookedneighbor
Copy link
Contributor

Actually, you opened the issue #4460

@Alys Alys changed the title Enter key should not post chat messages Enter/Return key should not post chat messages Sep 28, 2016
@Hus274
Copy link
Contributor

Hus274 commented Oct 3, 2016

I can take a look into this one.

@Hus274
Copy link
Contributor

Hus274 commented Oct 3, 2016

Submitted a pull request removing the send chat by clicking enter functionality.

crookedneighbor pushed a commit that referenced this issue Oct 7, 2016
…ine (#8096)

* changing behavior so hitting enter in a chat box only now inserts a newline instead of submitting the form. closes #8066

* Adding a tooltip message
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