feat: Revamp Google TV movie details with Compose - #3135
Open
cloudflix-x wants to merge 35 commits into
Open
Conversation
Author
|
The raw dumpsys gfxinfo logs are attached here: BENCHMARK_DUMPSYS_LOGS.md. Feel free to verify the performance on your side as well. |
…mponents, screens, and sample data
…omponent defaults
…nd edge-to-edge hero banner
…make about card focusable, and ensure full hero banner visibility
- CloneflixComposeTheme: expand theme tokens with new color palette entries - CloneflixBadgeComponents: adjust badge styling and layout tweaks - CloneflixMovieCard: refine card layout and visual treatment - CloneflixMovieInfoComponents: minor info layout improvements - CloneflixHeroBannersComposeScreen: hero banner UI enhancements - CloneflixMoviePreviewComposeScreen: preview screen polish - ResultFragmentTv: minor additions for TV result fragment
…cated Play Trailer button
…and row recycling
…eads, and stable models
…art resume, and adaptive provider outline
…ong-Click options dialog on TV
…een ViewModel and Compose Movie Details
…gn and Compose conventions
cloudflix-x
force-pushed
the
design-system-cloneflix
branch
from
August 27, 2026 03:31
cf8dd7c to
2a0a10d
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
#3129 - Migrated TV movie details screen (
ResultFragmentTv) from legacy XML to a modern, modular Jetpack Compose architecture (MovieDetailsComposeScreen). The UI has been broken down into decoupled components to improve maintainability and modernize the UX in line with leading streaming platforms, while preserving 100% of existing functionality and significantly optimizing memory and rendering performance.Key Highlights
fragment_result_tv.xmland monolithic view-binding with isolated, reusable Compose sections underui/result/compose/(Hero Banner, Episodes List, Season/Dub Dropdowns, Cast & Crew, Airing Schedule, and Recommendations), making the codebase cleaner and significantly easier to maintain.LazyColumnwith explicit keys/contentTypes), drastically flattening the view hierarchy and reducing memory overhead on TV hardware.Performance Benchmark Matrix
1. Overall Summary Comparison
2. Breakdown per Scenario
A. Initial Load & Fetching State (Worst Case)
B. Post-Fetch / Cached Scrolling State (Normal Use)
3. Key Takeaways
Validation
./gradlew :app:compileStableDebugKotlin./gradlew :app:assembleStableReleaseBENCHMARK_DUMPSYS_LOGS.mdAI Disclosure
Screenshoot
Hero Banner
Episodes & Recomendations
Video Preview
Match with app theme