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

Autofocus conversation text input when switching back to application #2885

Closed
wants to merge 1 commit into from

Conversation

kennu
Copy link

@kennu kennu commented Nov 5, 2018

First time contributor checklist:

Contributor checklist:

Description

Very simple fix to automatically focus the text input field of an open conversation when switching back to the Signal application. This prevents text being "lost" when you switch to the app and immediately start typing, without first manually clicking the input field to give it focus.

@scottnonnenberg-signal
Copy link
Contributor

I don't think this is going to work. For example, open the media gallery, switch away from a conversation, then switch back with this change.

@kennu
Copy link
Author

kennu commented Nov 5, 2018

I guess this would some real-world testing running the client connected to production network..

@kennu
Copy link
Author

kennu commented Nov 5, 2018

I added checks to see if a panel is open, or if the audio recorder view is open, to avoid taking the focus in those situations.

@kennu
Copy link
Author

kennu commented Nov 6, 2018

If this is not a good approach, another option would be to do it like WhatsApp (if you start typing without focus, it automatically focuses the text input) or Telegram (clicking history view doesn't take focus away from text input).

@scottnonnenberg-signal
Copy link
Contributor

Really, what I'm looking for is testing. Contributing.md should help you get a staging app up and running, and then you can do exhaustive testing on this. I don't think we can merge this until you've done testing and summarized your findings here.

@vrde
Copy link

vrde commented Sep 10, 2019

Related: #1416

@kennu
Copy link
Author

kennu commented Sep 10, 2019

Apologies for not having time to work on this. I still suffer from this every day while using Signal Desktop, as it randomly loses focus of the text input field.

@scottnonnenberg-signal
Copy link
Contributor

Closing stale pull requests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants