-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Only deactivate label if field isn't focused #5789
Conversation
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.
Great job finding the fix so quickly. LGTM
Feel free to self-merge once the E2E tests will pass and you will see it before me 😄 |
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
I think this PR may be the culprit for this issue #5904 |
Please note that the entry format for some fields is conveyed using placeholder only, and the placeholder text disappears once user enters any character into the field. It is mandatory to display the permanently visible instructions when user entry is required (WCAG SC 3.3.2). The issue is mentioned in #5703 (comment) |
🚀 Deployed to production by @roryabraham in version: 1.1.8-9 🚀
|
Details
This PR fixes an issue where the Expensitext label value will move over the cursor after deleting the content but keeping focus. In the case that the input has a label and a placeholder, this looks especially bad (see https://github.com/Expensify/Expensify/issues/180809).
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/180809
Tests/QA
Issue Corporate Cards
and thenConnect Bank Account
. Fill out anything for the values onConnect Bank Account
and submit to get to theCompany Information
stepCompany Information
step, the phone number placeholder doesn't overlap the label like in https://github.com/Expensify/Expensify/issues/180809 when you enter a value and then remove the valueTested On
Screenshots