-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
iPad - Chat - Popover menu animated from the bottom instead of from side #6529
Comments
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open
|
Triggered auto assignment to @deetergp ( |
This is the way the iOS app functions on iPhone. How is it that we expect that the iOS app should function like desktop & web on the iPad? I'm not sure I agree that this is a blocker. Thoughts @roryabraham @marcaaron @parasharrajat? |
This is working as expected. On bigger screens or landscape iPad, menu should animate from left You can easily evaluate it. If you are able see both LHN and report screen then menu should not be full screen and animation should be same as desktop. |
Great point! I just pulled down the production version of the app and it behaves the same way. So not a blocker. Closing! |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Issue was found when executing #6278
Action Performed:
Expected Result:
The Popover menu should appear the way it appears in Web and Desktop.(Animated from the side instead of from bottom)
Actual Result:
Popover menu animated from the bottom instead of from side
Workaround:
Unknown
Platform:
Where is this issue occurring?
Version Number: 1.1.17-1
Reproducible in staging?: Yes
Reproducible in production?: PR issue
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
Issue was found when executing #6278
Bug5350996_RPReplay_Final1638226171.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: