-
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
iOS - Request money - Green CTA button in BNP and confirmation page has different placement #30861
Comments
Triggered auto assignment to @kadiealexander ( |
Bug0 Triage Checklist (Main S/O)
|
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open
|
Triggered auto assignment to @joelbettner ( |
I'm not seeing problem here...those two button LOOK consistent to me. @shawnborton are you seeing anything here? @lanitochka17 can you state a little more clearly what is not consistent and perhaps show the difference in the way the button looks on staging vs production? Edit: is it the gap between the bottom of the button and the edge of the screen that is inconsistent? |
Posted this in Slack but might as well post it here as well: it looks like the numpad + button on the amount screen got shifted down somehow and is now too close to the bottom, which is then causing it to be in a different spot than the request button. Screenshots for reference: |
A PR fixing this issue is just merged #30887 |
Very nice @ishpaul777! I see that the changes in that PR were tested for Android. Were they tested for iOS as well? |
The changes got deployed to staging. I just tested it. All is good. I'm going to close this issue. |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 1.3.95-5
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
**Issue reported by:**Applause - Internal Team
Slack conversation:
Action Performed:
Expected Result:
The green button at the bottom should have a consistent placement in both BNP and confirmation page
Actual Result:
The green button at the bottom does not have a consistent placement in BNP and confirmation page
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6262524_1699077151165.staging.mp4
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: