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

reply box loses focus after sending #2097

Open
cfm opened this issue Jul 8, 2024 · 2 comments
Open

reply box loses focus after sending #2097

cfm opened this issue Jul 8, 2024 · 2 comments
Labels
bug Something isn't working release QA ux

Comments

@cfm
Copy link
Member

cfm commented Jul 8, 2024

Description

After sending a reply with <Ctrl>-<Enter>, the reply box loses focus.

Steps to Reproduce

  1. Type a reply.
  2. Press <Ctrl>-<Enter> to send.

Expected Behavior

The reply box regains focus (i.e., for a subsequent reply).

Actual Behavior

The reply box loses focus.

Comments

The chat UX idiom encourages people to (think they can) send multiple short messages in rapid succession, which is clumsy if the reply box can be submitted with the keyboard but must regain focus via the mouse.

@cfm cfm added bug Something isn't working ux release QA labels Jul 8, 2024
@rocodes
Copy link
Contributor

rocodes commented Jul 9, 2024

It looks like this may be intentional(ish) behaviour, or behaviour we haven't standardized on:

# Text area refocus flag.
self.refocus_after_sync = False

Originally I think this was introduced to avoid the replybox losing focus if a user was using the replybox while a background sync was applied. But a side-effect is the behaviour you mention (the replybox requires mouseclick or inconvenient tab/keyboard navigation in order to be refocused).

I think it's a fair point - it's inconvenient to get back to the replybox without using the cursor. I would say we should keep this open but not treat it as a release blocker - it's not a regression but we should put more time into all of our accessibility/navigation behaviours and making them consistent/predictable. That work for you @cfm ?

@cfm
Copy link
Member Author

cfm commented Jul 9, 2024

Oh, I absolutely agree that this is a long-standing UX inconsistency, neither a regression nor (remotely) a release blocker.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working release QA ux
Projects
None yet
Development

No branches or pull requests

2 participants