HomePhabricator
Diffusion Comm 92d6fa3e3161

[native] Minor updates to KeyserverSelection TextInput

Description

[native] Minor updates to KeyserverSelection TextInput

Summary:
While working on the TextInputs for UsernameSelection and PasswordSelection, I realized a couple updates I should make to the TextInput in KeyserverSelection:

  1. Set onSubmitEditing, so that the user can use the software keyboard action button to proceed to the next screen.
  2. Use returnKeyType="go" instead of the default "next". The latter matches the copy of our RegistrationButton, but is treated by the OS as more of a "next input field" button. Whereas "go" is meant for "I'm done with this page, go to the next one" and utilizes a design that indicates a more explicit action.

Test Plan:

  1. I tested onSubmitEditing by pressing the button while the TextInput was empty (nothing happens, same as before) and while the TextInput was populated (proceeds to the next screen).
  2. Here are screenshots for the returnKeyType change:
beforeafter
Screenshot 2023-05-25 at 3.03.49 PM.png (2×1 px, 929 KB)
Screenshot 2023-05-25 at 3.04.12 PM.png (2×1 px, 928 KB)

Reviewers: atul, ginsu

Reviewed By: atul

Subscribers: tomek

Differential Revision: https://phab.comm.dev/D7989

Details

Provenance
ashoatAuthored on May 25 2023, 11:59 AM
Reviewer
atul
Differential Revision
D7989: [native] Minor updates to KeyserverSelection TextInput
Parents
rCOMM9c414ebbc349: [native] Introduce PasswordSelection screen
Branches
Unknown
Tags
Unknown