-
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
[$250] Safari mWeb - Keyboard won't open for some inputs (New Chat, Change Password, New Room) #10414
Comments
Triggered auto assignment to @iwiznia ( |
Triggered auto assignment to @NicMendonca ( |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat ( |
Triggered auto assignment to @sketchydroide ( |
The text is not having top "priority" or the list is getting auto focus. You should give change the layout, something zIndex or ... set focus programmatically. |
Interesting.. it tested well when I added screenshots for PR #10194 |
I can't repro this bug on mWeb iOS, and mWeb Android. mWeb iOS Safari Screen.Recording.2022-08-25.at.1.12.34.AM.movmWeb Android Chrome screen-20220825-010957.mp4 |
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results. If a production regression has occurred a Root Cause Analysis is required. Please follow the instructions here. If a regression has occurred and you are the assigned CM follow the instructions here. If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future. |
@rushatgabhane are you recommending that I close this? |
@NicMendonca not exactly. Let's close this bug if @kbecciv cannot repro it either |
@rushatgabhane Issue is reproduced with latest build 1.1.94.4 RPReplay_Final1661904177.MP4 |
This is not yet solved. |
@kevinksullivan do you still think we should fix this one? And is it still an $8k bug? Just wanted to check in since it's been deprioritized since around October now. |
Related slack conversation: https://expensify.slack.com/archives/C01SKUP7QR0/p1707765399023319 |
|
Changing price in GH as this has not been a priority. @kbecciv can you please confirm you can reproduce this still? Then we can source volunteers if necessary. |
Can confirm this is still reproducible on mWeb Safari for some inputs Screen.Recording.2024-03-29.at.16.52.45.mov |
@kevinksullivan fyi I added #25477 (which is held on this issue) to the vip-vsb project and I think this should be added to the same project since it's chat-related functionality. |
Any updates here? |
Friendlly bump - any updates? |
There has been a lot of discussion on this issue. We had a PR to solve this which was abandoned #23922. I think we can open a fresh issue if we want to tackle this further. |
@parasharrajat are you suggesting that we discuss whether we want to still solve this or just close the issue? |
@sakluger I will loop you in as BZ here if you don't mind, as I'm going OOO for a bit. |
Sounds good. I'll ask @parasharrajat in Slack if we should close this issue. |
We're going to close out the issue. Given all the work that's been done, @parasharrajat is eligible for the full $250 payment. I'll post a payment summary below. |
Summarizing payment on this issue: Contributor+: @parasharrajat $250, please request on Newdot |
Payment requested as per #10414 (comment) |
$250 approved for @parasharrajat |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Issue found when executing PR #10194
Action Performed:
case 1
case 2
case 3
case 4
Expected Result:
Keyboard is open and cursor focused when entering the new screen
Actual Result:
Keyboard won't open
Workaround:
Unknown
Platform:
Where is this issue occurring?
Version Number: 1.1.88.13
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Case 1
Bug5691734_mWeb_10194.mp4
Case 2
Bug5773619_11587_mweb_ios.mp4
Case 3
RPReplay_Final1661701238.mov
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Tea
Slack conversation:
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: