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

More Chat UX Tweaks #2427

Closed
julianlam opened this issue Nov 19, 2014 · 2 comments
Closed

More Chat UX Tweaks #2427

julianlam opened this issue Nov 19, 2014 · 2 comments
Assignees
Milestone

Comments

@julianlam
Copy link
Member

... because I like doing this to @barisusakli :goberserk:

  1. When I hit enter, the keydown is captured by an event listener that sends the message. Add e.stopPropagation(); to this as well, because the default browser behaviour still happens, which adds a line break to the textarea 😄
  2. When a message is about to be sent, disable textarea+send button until event:chats.receive is received
  3. Whenever the textarea receives focus, move the cursor to the end
@barisusakli
Copy link
Member

Cant repro 1. When I click enter it sends the mesage and empties the text area.

2 Don't thin we should do this either, cant think of skype disabling my chat input everytime I send something.

3 If I do this then I can't go to the middle of a string in the chat.

@julianlam
Copy link
Member Author

  1. Try throttling your connection: Chrome Dev tools -> Hit the mobile icon:

selection_025

As for 2, good point. Can you detect whether the focus was from mouseclick? Maybe check e.which

@barisusakli barisusakli modified the milestones: 0.6.0, 0.6.1 Dec 16, 2014
@barisusakli barisusakli removed this from the 0.6.1 milestone Feb 4, 2015
@barisusakli barisusakli added this to the 0.8.2 milestone Sep 14, 2015
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

2 participants