-
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
[$500] mWeb/Safari - Edit message - Text input not focused & keypad doesn't appear when edit message box activated #32216
Comments
Job added to Upwork: https://www.upwork.com/jobs/~013eec6f943cbf57c0 |
Triggered auto assignment to @johncschuster ( |
Bug0 Triage Checklist (Main S/O)
|
Triggered auto assignment to Contributor-plus team member for initial proposal review - @getusha ( |
ProposalPlease re-state the problem that we are trying to solve in this issue.We want the text input to be focused and keyboard to be shown when the Edit Message button is pressed. What is the root cause of that problem?The issue is that we are not indicating that the text box should be focused (aside from the UI showing it is, the browser is not focusing it) when you click the menu item. We're specifically handling for mWeb/Safari in this specific issue as the DOM hasn't loaded at this point. What changes do you think we should make in order to solve the problem?On the initial load of the message edit component, we should add a check to make sure that the input is in the DOM and ready to be focused before we update the selection prop. We will also look for how other parts of the code interact with the input to see if it might be causing the input to lose focus after the selection prop is updated. |
Looks like a dupe of #10414, we can close it. |
This is not reproducible on the latest main |
Thanks for confirming that! I'll go ahead and close it. |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 1.4.5-3
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Expected Result:
As per step 4, the text input is focused and the keyboard appears for mobile when edit message box is activated
Actual Result:
Text input not focused & keypad doesn't appear when edit message box activated
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6294978_1701281839860.SYRC8491.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: