fix(ui): tighten media detail spacing
This commit is contained in:
@@ -3885,7 +3885,7 @@ class _MediaDetailScreenState extends State<MediaDetailScreen>
|
||||
const innerPadding = 3.0;
|
||||
final imageSize = cardWidth;
|
||||
// image + inner padding + text area + outer list padding + focus scale headroom
|
||||
final containerHeight = imageSize + innerPadding * 2 + 66 + 16;
|
||||
final containerHeight = imageSize + innerPadding * 2 + 58 + 10;
|
||||
|
||||
final theme = Theme.of(context);
|
||||
final actorNameStyle = theme.textTheme.bodyMedium?.copyWith(fontWeight: FontWeight.w600);
|
||||
@@ -3988,7 +3988,7 @@ class _MediaDetailScreenState extends State<MediaDetailScreen>
|
||||
final cardWidth = _getResponsiveCardWidth();
|
||||
// 16:9 aspect ratio for clip thumbnails (cardWidth includes 8px padding on each side)
|
||||
final posterHeight = (cardWidth - 16) * (9 / 16);
|
||||
final containerHeight = posterHeight + 66;
|
||||
final containerHeight = posterHeight + 52;
|
||||
|
||||
return Focus(
|
||||
focusNode: _extrasFocusNode,
|
||||
|
||||
@@ -68,6 +68,8 @@ class TvBrowseRailLayout {
|
||||
|
||||
static double railInteractionExpansionForScale(double scale) => (12 * scale).clamp(8, 18).toDouble();
|
||||
|
||||
static double itemGapForScale(double _) => 0;
|
||||
|
||||
static double hubStripHeightForScale(double scale) => 36 * scale;
|
||||
|
||||
static double hubStripGapForScale(double _) => 0;
|
||||
@@ -114,7 +116,7 @@ class TvBrowseRailLayout {
|
||||
}) {
|
||||
final focusExtra = FocusTheme.focusBorderWidth * 2 * scale;
|
||||
final railEdgePadding = focusExtra + (12 * scale);
|
||||
final itemGap = 8 * scale;
|
||||
final itemGap = itemGapForScale(scale);
|
||||
final isPersonHub = TvBrowseRailLayout.isPersonHub(hub);
|
||||
final hasWide = !isPersonHub && hub.items.any((item) => item.usesWideAspectRatio(episodePosterMode));
|
||||
final hasTall = !isPersonHub && hub.items.any((item) => !item.usesWideAspectRatio(episodePosterMode));
|
||||
|
||||
Reference in New Issue
Block a user