-
Notifications
You must be signed in to change notification settings - Fork 0
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
Landscape-Mode for RideView #335
Conversation
63e8b9b
to
eb665cf
Compare
What do you think about the design @PhilippMatthes ? |
I like it but the speedometer should fill the available vertical screen space |
End button should be on the left to make more space for the speedometer |
0dc156f
to
a7b08f4
Compare
wdym? |
To enable landscape-mode for iOS, we had to set a flag in XCode (not just implement it within the app). Maybe we need to do the same when compiling through XCodeCloud. |
Tested it on iPhone and Android. It works great. The only problem I found is that in landscape mode right now, we don't use the complete screen. On both, iPhone and Android, we are leaving out parts on the right and left that remain black. Is this caused by technical limitations of Flutter, or are we able to fix this? @kruegercharles |
I found another problem. Currently, in landscape mode, clicking on traffic signals (to center them in the view) does not work as desired. @kruegercharles |
One more problem: On Android, when not using energy-saving mode, there is a black bar at the top of the screen when using portrait mode (the map should use the whole screen, and the top system status bar should be visible). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See my comments above
I'm not sure but I would put this into another ticket. |
f821ad2
to
1e20869
Compare
Currently doesn't work on iOS. On the home view the rotation somehow is possible once after the app started (although the screen rotates back directly). The button idea was discarded? |
I've just tried it and it works on Thomas's iPhone. You have to enable landscape mode in XCode though: https://medium.com/pragmatic-programmers/set-the-supported-device-orientation-53b154385772 The button was discarded because we use the native landscape mode and in my understanding you can only force the landscape mode via button or use the native landscape mode via gyroscope, but not both. |
See: https://trello.com/c/GLAILbH2
TODO
After merging this PR in dev, we need to let the app build via XCodeCloud and then install via Testfight to test if XCodeCloud accepts the landscape-mode (or if we have to first set a flag somewhere)
Currently the landscape mode is buggy in iOS >= 16. This should be fixed when upgrading to a new Flutter Version (at least >= 3.10.6, preferably newer). We should keep this in mind.