68b210d231e5

test: pin settings deck fill-to-gutter on tall screens (v2.9.26)

Add settingsDeckHeight(1200dp) == 1164dp to SettingsStackTest. Against the old MAX_DECK_H*2 (924dp) cap this returned 924, so the assertion fails on the pre-fix code and passes now — pinning the Fold 7 fix. Corrected the stale comment that still described an upper bound.

time
2026-08-01 21:32 UTC · 2 months ago
machine
koh2
parent
2f6a24344c49
2 files VERSION+1 −1app/src/test/java/com/peachpasswords/ui/settings/SettingsStackTest.kt+6 −4
VERSION+1 −1
@@ -1 +1 @@1-2.9.251+2.9.26

view file

app/src/test/java/com/peachpasswords/ui/settings/SettingsStackTest.kt+6 −4
@@ -26,13 +26,15 @@2626     }2727 2828     @Test fun settings_deck_fills_available_height_instead_of_the_browsers_small_popup_cap() {29-        // The deck used to hard-cap at 462dp (a literal port of the browser's small30-        // popup), stranding a third of a phone screen as dead space below the stack.31-        // It now fills the available height (minus the bottom gutter) up to a much32-        // larger bound that only matters on very tall/tablet screens.29+        // The deck used to hard-cap (a literal port of the browser's small popup,30+        // later a MAX_DECK_H*2 bound), which stranded dead space below the stack and31+        // clipped the Pro card's buttons on tall panels like the Galaxy Z Fold 7. It32+        // now fills the available height minus the bottom gutter on every screen size.3333         assertEquals(864.dp, settingsDeckHeight(900.dp))3434         assertEquals(444.dp, settingsDeckHeight(480.dp))3535         assertEquals(0.dp, settingsDeckHeight(24.dp))36+        // Taller than the old 924dp cap allowed: still fills to the gutter, no cap.37+        assertEquals(1164.dp, settingsDeckHeight(1200.dp))3638     }3739 3840     // ── Drag gesture resolution ───────────────────────────────────────────────

view file