Page MenuHomePhabricator

[native] Delay PasswordSelection autoFocus on Android
ClosedPublic

Authored by ashoat on May 26 2023, 7:57 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 7, 4:22 PM
Unknown Object (File)
Feb 21 2024, 7:09 PM
Unknown Object (File)
Feb 21 2024, 6:33 PM
Unknown Object (File)
Feb 21 2024, 6:17 PM
Unknown Object (File)
Feb 21 2024, 6:14 PM
Unknown Object (File)
Feb 21 2024, 7:34 AM
Unknown Object (File)
Jan 11 2024, 4:57 PM
Unknown Object (File)
Dec 16 2023, 8:19 PM
Subscribers

Details

Summary

While playing around with 1Password on Android, I noticed that the autofill prompt didn't appear when we autoFocused the password TextInput in PasswordSelection. However, it would appear if the user subsequently pressed / selected the TextInput manually.

Test Plan

I tested release builds with various delays for the focus on two phones: my Pixel 6a, and the older Pixel 3 in the office. 250ms consistently worked for both, while 100ms did not.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ashoat added inline comments.
native/account/registration/password-selection.react.js
82 ↗(On Diff #27092)

It's okay to call this hook conditionally because the condition (whether the device is an Android device) is guaranteed to never change

atul added inline comments.
native/account/registration/password-selection.react.js
82 ↗(On Diff #27092)

Would it make sense to add a comment here (exact text as annotation) to make it clear to future readers?

(They could probably find the diff via git blame regardless, but wouldn't hurt?)

This revision is now accepted and ready to land.May 26 2023, 10:36 AM
This revision was landed with ongoing or failed builds.May 26 2023, 10:59 AM
This revision was automatically updated to reflect the committed changes.