App
App copied to clipboard
[$8000] Safari mWeb - Keyboard won't open for some inputs (New Chat, Change Password, New Room)
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Issue found when executing PR https://github.com/Expensify/App/pull/10194
Action Performed:
case 1
- Access staging.new.expensify.com
- Sign into a valid user
- Click on the "Global action" button (Big green plus button in LHN)
- Click New Chat
case 2
- Go to https://staging.new.expensify.com/
- Login with expensifail account
- Create a Workspace
- Tap "+"
- Tap "New Room"
case 3
- Go to https://staging.new.expensify.com/
- Sign into a valid user
- Go to settings > security > change password
case 4
- Go to https://staging.new.expensify.com/
- Sign into a valid user
- Go to some DM
- Open the in app emoji picker
- Select some emoji
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?
- Safari Mobile Web
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
https://user-images.githubusercontent.com/93399543/185005745-73a26290-2721-4b14-b3bd-ba0553a2a0a8.mp4
Case 2
https://user-images.githubusercontent.com/93399543/195383970-9354b2d0-cffc-4b1d-b5d2-2804c2462a5e.mp4
Case 3
https://user-images.githubusercontent.com/43995119/190952432-d072befe-c01d-404e-92fd-b6c71ff230c9.mov
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Tea
Slack conversation:
Triggered auto assignment to @iwiznia (Engineering
), see https://stackoverflow.com/c/expensify/questions/4319 for more details.
Triggered auto assignment to @NicMendonca (External
), see https://stackoverflow.com/c/expensify/questions/8582 for more details.
job post: https://www.upwork.com/jobs/~01f21a08ef50dd0283
Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat (Exported
)
Triggered auto assignment to @sketchydroide (Exported
), see https://stackoverflow.com/c/expensify/questions/7972 for more details.
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 https://github.com/Expensify/App/pull/10194
I can't repro this bug on mWeb iOS, and mWeb Android.
v 1.1.89-1
mWeb iOS Safari
https://user-images.githubusercontent.com/29673073/186508861-e30fecf7-d584-4118-9b00-083c5e661694.mov
mWeb Android Chrome
https://user-images.githubusercontent.com/29673073/186509173-15482710-c717-44f9-9e3e-46d97ee19b23.mp4
⚠️ Looks like this issue was linked to a possible regression on PRODUCTION here
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
https://user-images.githubusercontent.com/93399543/187564523-ea61b4d2-b8ff-4d5f-a70f-2d42605dc211.MP4
Still reproducible ☝️
@parasharrajat what should we do? (i still can't reproduce it)
I will check it later.
I am able to reproduce this. iPhone 13 (iOS 15). I feel like maybe this is happening due to the banner. The banner is not visible when a new chat page is opened and there is empty space below the modal. This empty seems equal to the banner height.
@parasharrajat did you have time to look into this?
I already did https://github.com/Expensify/App/issues/10414#issuecomment-1238555503. Are you talking about something else?
I am open to proposals.
no sorry, it's weird, but I' sure I did not have you comment with the screenshot when I wrote my comment, maybe GH did not refresh it, weird.
price doubled - https://www.upwork.com/jobs/~01f21a08ef50dd0283
doubled: https://www.upwork.com/jobs/~01f21a08ef50dd0283
⚠️ Looks like this issue was linked to a possible regression on PRODUCTION here
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.
doubled: https://www.upwork.com/jobs/~01f21a08ef50dd0283
Is the issue I'm facing the same as this one? On desktop, staging, v1.2.15-2
- Follow repro steps in OP
- After clicking a new chat, cursor isn't focused on compose box, unable to start typing immediately, have to click compose box to do so.
Same for when I click on a chat in LHN, cursor doesn't show in compose box
That sounds like a new issue and I saw a bug report for it somewhere. This issue is concerned on the search box on the new chat and new group page.
Thanks @parasharrajat https://expensify.slack.com/archives/C01GTK53T8Q/p1665792005925189
doubled price: https://www.upwork.com/jobs/~01f21a08ef50dd0283
@NicMendonca Should we merge all of the mentioned issues into this https://github.com/Expensify/App/pull/11794#issuecomment-1280776271. cc: @sketchydroide
@parasharrajat maybe open up a discussion in slack?
@sketchydroide What do you think of https://github.com/Expensify/App/issues/10414#issuecomment-1285514867?