Skip to content

[$250] iOS/Android - Workspace - Keypad overlaps "Call me" button #9807

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

Closed
kbecciv opened this issue Jul 8, 2022 · 24 comments
Closed

[$250] iOS/Android - Workspace - Keypad overlaps "Call me" button #9807

kbecciv opened this issue Jul 8, 2022 · 24 comments
Assignees
Labels
Engineering Hourly KSv2 Reviewing Has a PR in review

Comments

@kbecciv
Copy link

kbecciv commented Jul 8, 2022

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Action Performed:

  1. Launch the app and login
  2. Tap profile and select workspace
  3. Tap "?" and tap "Request a set up call"
  4. Tap phone number field and observe

Expected Result:

Opened keypad should not overlap the "Call me" button

Actual Result:

Keypad overlaps "Call me" button

Workaround:

Unknown

Platform:

Where is this issue occurring?

  • iOS
  • Android

Version Number: 1.1.82.5

Reproducible in staging?: Yes

Reproducible in production?: No

Email or phone of affected tester (no customers): any

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

@melvin-bot
Copy link

melvin-bot bot commented Jul 8, 2022

Triggered auto assignment to @alex-mechler (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@alex-mechler
Copy link
Contributor

Display issue, looks like a perfect external issue!

@alex-mechler alex-mechler removed their assignment Jul 8, 2022
@alex-mechler alex-mechler added the External Added to denote the issue can be worked on by a contributor label Jul 8, 2022
@melvin-bot
Copy link

melvin-bot bot commented Jul 8, 2022

Triggered auto assignment to @adelekennedy (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@melvin-bot
Copy link

melvin-bot bot commented Jul 11, 2022

@adelekennedy Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@adelekennedy
Copy link

internal
external

@melvin-bot melvin-bot bot removed the Overdue label Jul 12, 2022
@parasharrajat
Copy link
Member

Maybe a regression from #9665.

@WtfLaika
Copy link

WtfLaika commented Jul 12, 2022

Have you tried to use KeyboardAvoidingView? Wrap all components with KeyboardAvoidingView set behavior='position' and make small padding bottom or set value for keyboardVerticalOffset.

@parasharrajat
Copy link
Member

@adelekennedy It is missing Exported label.

@WtfLaika
Copy link

WtfLaika commented Jul 12, 2022

Must be something like this
Screenshot 2022-07-12 at 22 19 05

@melvin-bot
Copy link

melvin-bot bot commented Jul 14, 2022

Triggered auto assignment to Contributor-plus team member for initial proposal review - @mananjadhav (Exported)

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Jul 14, 2022
@melvin-bot
Copy link

melvin-bot bot commented Jul 14, 2022

Triggered auto assignment to @mountiny (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@melvin-bot melvin-bot bot changed the title iOS/Android - Workspace - Keypad overlaps "Call me" button [$250] iOS/Android - Workspace - Keypad overlaps "Call me" button Jul 14, 2022
@mananjadhav
Copy link
Collaborator

I haven't been able to reproduce this on my iOS v1.1.78-8.

call-me-button-position_DJYClIFB.mp4

but can on v1.1.82-5

image

As @parasharrajat pointed out it looks like a regression.

@mountiny
Copy link
Contributor

@kbecciv Can you please confirm if you can repro this in production as well as staging? That would help us narrow down the problem.

@kbecciv
Copy link
Author

kbecciv commented Jul 16, 2022

@mountiny I checked production version and issue is not there. Let's add Deploy blocker to this issue.

Here is potential PR #9573

Image from iOS (40)

@kbecciv kbecciv added the DeployBlockerCash This issue or pull request should block deployment label Jul 16, 2022
@github-actions github-actions bot added Hourly KSv2 and removed Weekly KSv2 labels Jul 16, 2022
@OSBotify
Copy link
Contributor

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.

@mountiny
Copy link
Contributor

@kbecciv Thanks for the confirmation, very helpful 🙌

Having a look now!

@mountiny
Copy link
Contributor

@kbecciv Could you actually confirm with a video share this really is not reproducible on production? The screenshot is not clear to me.

The reason I am asking is that it seems that this PR is the main suspect for causing it, however, it is already in production. I could not find anything in Staging which might be causing this.

@mountiny
Copy link
Contributor

K, this was a bit of pain to investigate.

### What happened

@ctkochan22 has introduced refactor of the FormAlertWithSubmitButton component in this PR. Those changes are fine on their own, however, then @yuwenmemon has started to use this component at the RequestCallPage in this PR which got merged right when the Form refactor happened so Yuwen could not really well know about those changes.

Additionally, given both of those PRs are already in production, it was probably this change of the styles, which seemingly should not have an effect on this behaviour, but it evidently did, but honestly i dont know right now why.

I am still not 100% sure how come it is fine in production and broken in staging, however, the PR I put up is fixing this problem by resetting the flex property on the form submit button on the RequestCallPage :)

@mountiny mountiny removed External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors labels Jul 17, 2022
@mountiny
Copy link
Contributor

Not External anymore, @adelekennedy would you be able to please delete the job from upwork for this one. Thank you very much!

@kbecciv
Copy link
Author

kbecciv commented Jul 17, 2022

@mountiny Please attached video from production per your request.

Image.from.iOS.40.MP4

@b1tjoy
Copy link
Contributor

b1tjoy commented Jul 17, 2022

Additionally, given both of those PRs are already in production, it was probably this change of the styles, which seemingly should not have an effect on this behaviour, but it evidently did, but honestly i dont know right now why.

@mountiny PR #9865 does not cause this behavior. FixedFooter is a View with style of flexShrink: 0, and the inner FormAlertWrapper is a View with style of flex: 1, the combination cause this issue, and reset the inner component style to flex: undefined will fix it, as the PR you just submitted :)

@ctkochan22
Copy link
Contributor

Great find @mountiny

@mountiny
Copy link
Contributor

@ctkochan22 Cheers 🙏

@kbecciv Thank you very much 🙌

@b1tjoy Thanks for follow up, I agree it is not the PR itself, but I am not sure why is this fine in production? Looking at the production code, the RequestCallPage and FormAlertWrapper look basically same (just different way of passing the styles, but I feel like that should not be a problem). The FixedFooter styles are same too.

@melvin-bot melvin-bot bot closed this as completed Jul 18, 2022
@ctkochan22
Copy link
Contributor

IMG_692B26768F60-1
Tested on staging ios, we are good

@chiragsalian chiragsalian removed the DeployBlockerCash This issue or pull request should block deployment label Jul 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Engineering Hourly KSv2 Reviewing Has a PR in review
Projects
None yet
Development

No branches or pull requests