Section: bump section item minHeight by 6dp (50 → 56)

User asked for taller settings rows. Bump the default minHeight in
all four SectionItemView family functions from DEFAULT_MIN_SECTION_ITEM_HEIGHT
(50dp) to DEFAULT_MIN_SECTION_ITEM_HEIGHT + 6.dp (56dp).

Scoped to SectionItemView callers only — does not touch the global
DEFAULT_MIN_SECTION_ITEM_HEIGHT constant, so non-section callers
(ChatItemInfoView, ComposeContextProfilePicker, TagListView,
UserPicker) keep the 50dp baseline.

Callers that pass explicit minHeight (e.g. 54dp in GroupChatInfoView
members) are unaffected.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
another-simple-pixel
2026-05-16 11:57:33 -07:00
parent 0c6ec43a8d
commit f9b4c97d3f
@@ -173,7 +173,7 @@ fun <T> SectionViewSelectableCards(
@Composable
fun SectionItemView(
click: (() -> Unit)? = null,
minHeight: Dp = DEFAULT_MIN_SECTION_ITEM_HEIGHT,
minHeight: Dp = DEFAULT_MIN_SECTION_ITEM_HEIGHT + 6.dp,
disabled: Boolean = false,
extraPadding: Boolean = false,
padding: PaddingValues = if (extraPadding)
@@ -197,7 +197,7 @@ fun SectionItemView(
@Composable
fun SectionItemViewWithoutMinPadding(
click: (() -> Unit)? = null,
minHeight: Dp = DEFAULT_MIN_SECTION_ITEM_HEIGHT,
minHeight: Dp = DEFAULT_MIN_SECTION_ITEM_HEIGHT + 6.dp,
disabled: Boolean = false,
extraPadding: Boolean = false,
padding: PaddingValues = if (extraPadding)
@@ -213,7 +213,7 @@ fun SectionItemViewWithoutMinPadding(
fun SectionItemViewLongClickable(
click: () -> Unit,
longClick: () -> Unit,
minHeight: Dp = DEFAULT_MIN_SECTION_ITEM_HEIGHT,
minHeight: Dp = DEFAULT_MIN_SECTION_ITEM_HEIGHT + 6.dp,
disabled: Boolean = false,
extraPadding: Boolean = false,
padding: PaddingValues = if (extraPadding)
@@ -242,7 +242,7 @@ fun SectionItemViewLongClickable(
fun SectionItemViewSpaceBetween(
click: (() -> Unit)? = null,
onLongClick: (() -> Unit)? = null,
minHeight: Dp = DEFAULT_MIN_SECTION_ITEM_HEIGHT,
minHeight: Dp = DEFAULT_MIN_SECTION_ITEM_HEIGHT + 6.dp,
padding: PaddingValues = PaddingValues(horizontal = CARD_ITEM_PADDING),
disabled: Boolean = false,
content: (@Composable RowScope.() -> Unit)