-
Notifications
You must be signed in to change notification settings - Fork 3k
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] Hybrid App - Spacing above keyboard and status bar overlap on deeplinking to user with no chat #53571
Comments
Triggered auto assignment to @jasperhuangg ( |
Triggered auto assignment to @mallenexpensify ( |
💬 A slack conversation has been started in #expensify-open-source |
👋 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:
|
Added 2 Screen_Recording_20241204_171047_Chrome.mp4Screen_Recording_20241204_171101_Chrome.mp4Screen_Recording_20241204_181915_Expensify.mp4 |
@izarutskaya It's not clear to me what the regression is. Is the keyboard taking up more space than it needs to? |
@jasperhuangg sorry for confusing, video for some reason not downloaded. We about the empty space above the keyboard. Updated GH ticket Screen_Recording_20241204_145106_Expensify.mp4 |
Definitely seems like a front-end issue |
Job added to Upwork: https://www.upwork.com/jobs/~021864380806227249536 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @paultsimura ( |
EDIT: I don't think an AdHoc build will be testable since it is HybridApp |
Minor display bug, probably not worth blocking deploy on. |
cc @jnowakow this might be related to stuff you're working on, FYI |
@jasperhuangg I'll have a look on it |
@izarutskaya is this a thing that you can reliably reproduce each time? |
@jasperhuangg I forgot to ask you to assign me here. I think I found the solution. |
Okey I thought I have solution for this one but it solves #53185. But when I try to reproduce this issue I get something like this (it's recording from standalone ND): ND-main.mov |
It seems that |
@jasperhuangg I'm not able to reproduce it anymore because I'm being navigated to the screen two times. Anyone who I asked doesn't see this behaviour so I suppose it's random problem on my side. |
Thanks @jnowakow , threw |
This issue has not been updated in over 15 days. @mallenexpensify, @jasperhuangg eroding to Monthly issue. P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do! |
Issue not reproducible during KI retests. (First week) |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: v9.0.71-0
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: No, reproducible on hybrid only
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5300573
Email or phone of affected tester (no customers): N/A
Issue reported by: Applause Internal Team
Action Performed:
Expected Result:
Header of the DM should be displayed below device status bar and regular spacing above keyboard should be visible
Actual Result:
There is huge spacing above keyboard, Say hello! message overlap status bar, and the header is not visible.
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6684248_1733314550526.Record_2024-12-04-12-55-01.1.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @paultsimuraThe text was updated successfully, but these errors were encountered: