-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Workspace settings - App's orientation does not change to Landscape #6711
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
Comments
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open
|
Triggered auto assignment to @mountiny ( |
Triggered auto assignment to @MitchExpensify ( |
Removing deploy blocker as it does not seem very urgent. However, I have teated on staging and the I cant switch to landscape at any screen. |
This is intentional after this PR #6608. I think we can close this issue, unless not being able to change to landscape mode is blocking some functionality. |
@luacmartins thank you for a heads up! Is there an official way to update QA tests so they test for not being able to turn to landscape instead? 😅 |
Based on the Slack thread linked in the issue by Carlos, also here, this is intentional and on phones, we should not be able to rotate to landscape anymore. However, the rotation should be possible on tablets. @kavimuru could you please update the testing instructions to make sure we correctly QA for this new expected behaviour? Thank you very much 🙇 |
@mountiny from the accessibility perspective restricting the content view and operation to a single display orientation fails WCAG 2.1 SC 1.3.4 |
@ogumen ah, thank you for raising this issue. It has been a decision made by more people so i will need to discuss with them. |
@mountiny, I have edited the steps. Also it works fine in iPad though. CTUV5023.MP4. |
Thank you @kavimuru! I will bring up the accessibility issues in Slack to see what is our approach here and whether to revert this decision now knowing it breaks accessibility rules. |
Discussion still going on. |
The teams are still discussing this coming to Christmas break. |
@kavimuru We have come to conclusion we will keep it as is, therefore on mobile, we will only have portrait. On tablet, we will have both portrait and landscape. We might come back to this issue later. cc @AndrewGable |
Gonna close it for now, feel free to reopen anytime. |
Uh oh!
There was an error while loading. Please reload this page.
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Issue was found when executing #6648
Action Performed:
Expected Result:
App's orientation changes to landscape
Actual Result:
App's orientation remains in Portrait
Workaround:
Unknown
Platform:
Where is this issue occurring?
Version Number: 1.1.20-0
Reproducible in staging?: Yes
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
Image.from.iOS.MP4
Issue was found when executing #6648
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: