Skip to content

fix(mobile): compact details backdrop in phone landscape - #608

Merged
ProdigyV21 merged 2 commits into
ProdigyV21:mainfrom
Saelon600:fix/details-landscape-backdrop
Aug 25, 2026
Merged

fix(mobile): compact details backdrop in phone landscape#608
ProdigyV21 merged 2 commits into
ProdigyV21:mainfrom
Saelon600:fix/details-landscape-backdrop

Conversation

@Saelon600

Copy link
Copy Markdown

Summary

  • Reduces the media detail page backdrop height in phone landscape to 35% of screen height (capped at 200dp), instead of the previous 400dp floor which consumed the entire screen in landscape.
  • Applies the same compact height to the SkeletonLoader placeholder so the loading state matches.
  • Portrait, tablet, and TV layouts are unchanged.

Why

On phones in landscape, the coerceAtLeast(400.dp) floor on backdrop height exceeded the short edge of the screen, pushing all metadata, actions, and cast below the fold. This is the same class of issue as the merged compact bottom navigation (#599) and compact Live TV guide (#600).

Testing

  • Phone landscape: backdrop uses compact height (35%, max 200dp).
  • Phone portrait: backdrop retains existing 53% / 400dp floor.
  • Tablet/TV: unaffected (non-mobile layout path).
  • ./gradlew :app:compileSideloadDebugKotlin passes.
  • git diff --check passes.

Scope

Mobile details screen and skeleton loader only. No IPTV, EPG, player, backend, or app-version changes.

In landscape, the media detail page backdrop was forced to at least 400dp
by coerceAtLeast, which exceeded the entire screen height on phones.
Use 35% of screen height (capped at 200dp) in landscape, keeping the
existing 53%/400dp floor for portrait. Apply the same fix to the
SkeletonLoader placeholder.
@github-actions github-actions Bot added the area: android Changes to the Android app or Gradle build label Aug 25, 2026
@ProdigyV21
ProdigyV21 merged commit 83252ca into ProdigyV21:main Aug 25, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: android Changes to the Android app or Gradle build

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants