Skip to content
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

Android - New Room -The keyboard overlays the "New Room" button #13152

Closed
kbecciv opened this issue Nov 29, 2022 · 7 comments
Closed

Android - New Room -The keyboard overlays the "New Room" button #13152

kbecciv opened this issue Nov 29, 2022 · 7 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Internal Requires API changes or must be handled by Expensify staff

Comments

@kbecciv
Copy link

kbecciv commented Nov 29, 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. Open the App
  2. Login with any Expensifail account
  3. Click on FAB button -> New Room

Expected Result:

The keyboard must NOT overlays the "New Room" button

Actual Result:

The keyboard overlays the "New Room" button

Workaround:

Unknown

Platform:

Where is this issue occurring?

  • Android

Version Number: 1.2.33.2

Reproducible in staging?: Yes

Reproducible in production?: Yes

Email or phone of affected tester (no customers):

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

Notes/Photos/Videos: Any additional supporting documentation

Bug5841218_Record_2022-11-29-10-11-58_4f9154176b47c00da84e32064abf1c48.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

@kbecciv kbecciv added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Nov 29, 2022
@melvin-bot
Copy link

melvin-bot bot commented Nov 29, 2022

Triggered auto assignment to @tjferriss (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@tjferriss
Copy link
Contributor

tjferriss commented Nov 29, 2022

I've asked for help in #expensify-open-source with reproducing this on Android.

@srikarparsi
Copy link
Contributor

I can take this one @tjferriss, I think we just need to wrap it in a scroll view but I'll investigate further

@srikarparsi srikarparsi self-assigned this Nov 30, 2022
@tjferriss tjferriss added the Internal Requires API changes or must be handled by Expensify staff label Nov 30, 2022
@marcaaron
Copy link
Contributor

Should this be getting tracked with the other KeyboardAvoidingView issues? #10273 cc @tgolen

@melvin-bot melvin-bot bot added the Overdue label Dec 2, 2022
@srikarparsi
Copy link
Contributor

I'm not able to replicate it in production:

Screen Shot 2022-12-04 at 6 15 27 PM

@kbecciv can you check if you're still able to reproduce this?

@melvin-bot melvin-bot bot removed the Overdue label Dec 4, 2022
@0xmiros
Copy link
Contributor

0xmiros commented Dec 4, 2022

This should be fixed by android keyboard refactor in #13106

@tjferriss
Copy link
Contributor

Tested via BrowserStack and it appears the bug is resolved so I'm going to close the issue.

Screen Shot 2022-12-05 at 1 50 15 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Internal Requires API changes or must be handled by Expensify staff
Projects
None yet
Development

No branches or pull requests

5 participants