-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Fix ExpensiInput autoFocus & tab Focus #4593
Conversation
@roryabraham do we need to apply the |
@francoisl Yep! |
|
Actually JK I am going to remove the CP label in this case to help me finish testing #4274 for the case of manual CP's |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So I tested this on iOS Safari and also desktop and it seems to work as expected 👍
However, I think we maybe should leave a comment explaining why we are not using the autoFocus
prop, to prevent someone coming along and undoing this in the future. NAB though.
Co-authored-by: Rory Abraham <47436092+roryabraham@users.noreply.github.com>
@roryabraham Your suggestions are up. Thanks. |
I am going to merge this w/o waiting for E2E because I want to test the CP audit trail and because the only change between the previous commit, which passed all checks, is a comment. |
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
CPing this to staging here: https://github.com/Expensify/App/actions/runs/1125331217 |
Fix ExpensiInput autoFocus & tab Focus (cherry picked from commit ff62078)
Staging deploy is running here, so hopefully once that finishes we'll see a comment on this PR to the tune of: "CPed to staging by @roryabraham..." |
🚀 Cherry-picked to staging by @roryabraham in version: 1.0.85-3 🚀
|
This has been deployed to production and is now subject to a 7-day regression period. |
🚀 Deployed to staging by @roryabraham in version: 1.0.85-10 🚀
|
This has been deployed to production and is now subject to a 7-day regression period. |
🚀 Deployed to production by @roryabraham in version: 1.0.86-11 🚀
|
This has been deployed to production and is now subject to a 7-day regression period. |
Details
Fixed Issues
$ Fixes #4514
Tests | QA Steps
Tested On
Screenshots
Web
focus-fix.mp4
Mobile Web
Desktop
iOS
Android