Skip to content
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

Support for landscape orientation #43

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

pupitetris
Copy link
Contributor

I use my tablet in landscape mode all of the time, supported on top of the keyboard. Applications that don't support a landscape orientation can be quite irritating to me, so I enabled the landscape orientation on the activites and wrote an alternate layout for a landscape orientation on big screens to make better use of the screen real estate. The other views look fine on landscape, so no changes were needed there.

Screenshot_20241222_124930_wseemann media romote

A pending issue is a remote controller layout that works well with small screens on landscape. I suppose we would have to get rid of the tab switcher (or make it vertical) and the top title to maximize the area available for the buttons.

as it is limiting activities to a portrait orientation, but
they are working acceptably for landscape orientation as well.

Orientation configuration changes should be managed correctly; as
the application stands, navigation flips to the Devices tab when
orientation changes, instead of persisting in the current tab.
Splits the buttons in two clusters. Right cluster mimics the original
Roku navigation cluster. Left cluster groups tv/media and
unconventional or elite buttons.
Kind of hackish:
https://stackoverflow.com/a/5317727
and
https://stackoverflow.com/a/23363016

So, now that a horizontal layout has been added, when the orientation
changes, we retain the selected tab/item.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant