-
-
Notifications
You must be signed in to change notification settings - Fork 3
Operator Setup tab rail is not scrollable — System clipped on small phones #134
Copy link
Copy link
Open
Labels
P1High priority — important for current cycleHigh priority — important for current cyclearea:androidAndroid / Jetpack Compose workAndroid / Jetpack Compose workarea:iosiOS SwiftUI shell (ios/OpenPocketCine)iOS SwiftUI shell (ios/OpenPocketCine)area:uiGeneral UI/UX, layout, design systemGeneral UI/UX, layout, design systembugSomething isn't workingSomething isn't workingreadyTriaged and ready to pick upTriaged and ready to pick up
Description
Metadata
Metadata
Assignees
Labels
P1High priority — important for current cycleHigh priority — important for current cyclearea:androidAndroid / Jetpack Compose workAndroid / Jetpack Compose workarea:iosiOS SwiftUI shell (ios/OpenPocketCine)iOS SwiftUI shell (ios/OpenPocketCine)area:uiGeneral UI/UX, layout, design systemGeneral UI/UX, layout, design systembugSomething isn't workingSomething isn't workingreadyTriaged and ready to pick upTriaged and ready to pick up
Maintainer + testers: on smaller phones, Operator Setup’s System tab is gone. You have to rotate to portrait to reach it.
What went wrong?
Seven tabs (Link → Sharing → View Assist → Controls → Display → Storage → System). Landscape uses a vertical rail that does not scroll. Each row is 43 pt + 5 pt gap (≈343 pt plus padding). Landscape height on SE / Mini / many phones is ~375 pt after the Operator Setup header and safe area — Storage/System clip off the bottom. Portrait already uses a horizontal
ScrollView/horizontalScroll, so rotating “to vertical” is the workaround.Already in tree
iOS
SettingsRootView:settingsTabStrip—ScrollView(.horizontal)settingsRail—VStackof tabs,.frame(maxHeight: .infinity, alignment: .top), noScrollViewAndroid
OperatorSetup.kt:SettingsTabStrip—horizontalScrollSettingsTabRail—Column(fillMaxHeight), noverticalScrollContent panes already scroll (
SettingsTabScrollArea/verticalScroll+ScrollMoreCue). The tab list does not.Do
ScrollView(indicators optional). System must be reachable on a 375-pt-tall landscape (SE-class) without rotating.SettingsTabRail: sameverticalScroll. Cue if the rail can scroll (content already hasScrollMoreCue).Out of scope
Dropping tabs. Moving System into another pane. Changing the seven-tab set.
Done when
Physical small phone, landscape: you can scroll the rail to System (legal / GitHub / keep screen awake). Portrait still works. Both shells.
Verify
Physical iPhone SE-class or Mini, landscape Operator Setup from live and from home. Android small/short landscape.
just native-check/just android-checkfor the layout test.