Skip to content

Redesign watchlist with Movies and Series rows#287

Merged
ProdigyV21 merged 1 commit into
ProdigyV21:mainfrom
chillpill244:watchlist_split
Jun 1, 2026
Merged

Redesign watchlist with Movies and Series rows#287
ProdigyV21 merged 1 commit into
ProdigyV21:mainfrom
chillpill244:watchlist_split

Conversation

@chillpill244
Copy link
Copy Markdown
Contributor

@chillpill244 chillpill244 commented Jun 1, 2026

Summary

  • Replaces the flat grid with two horizontal rows — Movies and Series — split by media type
  • WatchlistUiState now holds separate movies and series lists; all state emissions updated accordingly
  • Manual D-pad focus tracking (isFocusedOverride / enableSystemFocus = false) for reliable TV remote navigation
  • Long-press Enter removes an item; a -1L sentinel on enterKeyDownTimeMs prevents spurious removal when navigating to the screen
  • Mobile: larger cards, 48dp top padding, focus highlight suppressed on touch devices
  • TV: spacing tuned so both rows fit on screen; contentPadding on LazyRow prevents clipping of focused cards at the left edge

Test plan

  • Open watchlist on TV — Movies and Series rows appear, D-pad navigates correctly between rows and to the top bar
  • Navigate to watchlist from another screen — first item is not removed on entry
  • Open watchlist on mobile — no focus highlight, cards and headings are appropriately sized
  • Watchlist with only movies or only series — single row renders correctly
  • Empty watchlist — empty state shown with bookmark icon

🤖 Generated with Claude Code

Replaces the flat grid with two horizontal LazyRow sections split by
media type. Includes manual D-pad focus tracking, long-press removal,
mobile-specific sizing, and a KeyUp guard to prevent spurious deletion
on screen entry.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@chillpill244
Copy link
Copy Markdown
Contributor Author

image

@ProdigyV21 ProdigyV21 merged commit c7b8fbc into ProdigyV21:main Jun 1, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants