-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
mWeb/Safari - Security - Close account button hidden by keyboard #7572
Comments
Triggered auto assignment to @TomatoToaster ( |
I have a feeling that this is a limitation of how the keyboard works in safari mWeb, but I'm not entirely sure. I'll ask somewhere to double check if we want to fix this. |
Ah ok I see on iOS though that it fully brings the button up, it's a much better experience. Going to mark this as external and see if someone has a solution for this. |
Triggered auto assignment to @adelekennedy ( |
6 days overdue. This is scarier than being forced to listen to Vogon poetry! |
6 days overdue. This is scarier than being forced to listen to Vogon poetry! |
Still overdue 6 days?! Let's take care of this! |
8 days overdue is a lot. Should this be a Weekly issue? If so, feel free to change it! |
Oh just seeing this - did you mean to unassign me @TomatoToaster or should I open a job for this? |
Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Eep! 4 days overdue now. Issues have feelings too... |
Still overdue 6 days?! Let's take care of this! |
12 days overdue now... This issue's end is nigh! |
This issue has not been updated in over 14 days. eroding to Weekly issue. |
Triggered auto assignment to @puneetlath ( |
Assigning over to @JmillsExpensify since this is part of the Keyboard project. |
Thanks, I added this to the related tracking issue and I've put this issue on hold. I'll work on retesting everything once Tim's PRs hit staging. |
@JmillsExpensify Eep! 4 days overdue now. Issues have feelings too... |
Still on hold pending the linked issue. |
This is no longer on hold and can have proposals created for it with the new code. |
I actually can't reproduce this issue anymore. |
This was the flow I followed:
As expected, the close account button appears above the keyboard. |
I'm going to close this issue as a result. |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Expected Result:
''Close account'' button should be fully visible
Actual Result:
''Close account'' button hidden by keyboard
Workaround:
Unknown
Platform:
Where is this issue occurring?
Version Number: 1.1.36.0
Reproducible in staging?: Yes
Reproducible in production?: Yes
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Bug5438389_mWeb_7512-0402.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause
Slack conversation:
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: