From 7b93265b4ff91bc824ec4b873c821aa1aa02c993 Mon Sep 17 00:00:00 2001 From: Linus Rath <139418639+rathlinus@users.noreply.github.com> Date: Thu, 15 Oct 2026 03:21:18 +0200 Subject: [PATCH] fix: stop text fields cutting off their text at a larger font size --- src/components/Input.tsx | 2 +- src/screens/EmailListScreen.tsx | 8 ++++++-- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/src/components/Input.tsx b/src/components/Input.tsx index 40ecccf8..ca64524e 100644 --- a/src/components/Input.tsx +++ b/src/components/Input.tsx @@ -71,7 +71,7 @@ function makeStyles(c: ThemePalette) { inputWrapper: { flexDirection: 'row', alignItems: 'center', - height: componentSizes.inputHeight, // h-10 = 40px + minHeight: componentSizes.inputHeight, // h-10 = 40px; grows with a larger system font borderWidth: 1, borderColor: c.border, // border-input borderRadius: radius.sm, // rounded-md = 6px diff --git a/src/screens/EmailListScreen.tsx b/src/screens/EmailListScreen.tsx index 446f0399..f991a872 100644 --- a/src/screens/EmailListScreen.tsx +++ b/src/screens/EmailListScreen.tsx @@ -1985,14 +1985,18 @@ function makeStyles(c: ThemePalette) { flexDirection: 'row', alignItems: 'center', paddingHorizontal: spacing.md, - height: componentSizes.inputHeight, // h-10 = 40px + // A minimum, not a fixed height: at a larger system font the text used + // to be cut off at the top. + minHeight: componentSizes.inputHeight, // h-10 = 40px backgroundColor: c.surface, borderRadius: radius.sm, borderWidth: 1, borderColor: c.border, gap: spacing.sm, }, - searchInput: { flex: 1, ...typography.body, color: c.text }, + // No lineHeight and no Android default padding on a TextInput: both clip + // the text when the system font is scaled up. + searchInput: { flex: 1, fontSize: typography.body.fontSize, color: c.text, paddingVertical: 4 }, filterButton: { width: componentSizes.buttonMd, height: componentSizes.buttonMd, alignItems: 'center', justifyContent: 'center',