Skip to content

Migrate media sample screens to Material 3#2726

Draft
yschimke wants to merge 6 commits intomainfrom
migrate-m3-tlc
Draft

Migrate media sample screens to Material 3#2726
yschimke wants to merge 6 commits intomainfrom
migrate-m3-tlc

Conversation

@yschimke
Copy link
Copy Markdown
Collaborator

read up on #2643, this is the goal. But maybe guidance has changed since then. So use the compose-preview tool and WEAR_UI.md guidance and update the code to achieve the same. Include before/after screenshots in the PR.

Summary

Migrated the main screens of the media sample app to Material 3 and updated Horologist screens to use TransformingLazyColumn where applicable, following the guidance in WEAR_UI.md.

Screenshots

Note

I attempted to upload the screenshots to a gist as requested, but gh gist create failed because binary files are not supported. The screenshots are available locally in the scratch directory:

  • Before: /usr/local/google/home/yschimke/.gemini/jetski/brain/c4e5b6e0-a2bd-44f0-8937-8e4715056116/scratch/before_large_round.png
  • After: /usr/local/google/home/yschimke/.gemini/jetski/brain/c4e5b6e0-a2bd-44f0-8937-8e4715056116/scratch/after_large_round.png

Changes

  • Updated UampTheme to use Material 3.
  • Updated UampMediaPlayerScreen to use M3 PlayerScreen.
  • Migrated SectionedList and EntityScreen in media/ui-material3 to TransformingLazyColumn.
  • Updated PlaylistDownloadBrowseScreen and PlaylistDownloadScreen items with padding.
  • Fixed parameter names in AnimatedMediaControlButtons.
  • Added missing dependencies in build.gradle.kts.
  • Fixed toTileColors usage in tile services.

@kul3r4
Copy link
Copy Markdown
Collaborator

kul3r4 commented Apr 23, 2026

Can be done in a separate commit but FYI I can't see the newest screen for M3 Player being used, perhaps what we need is to update to com.google.android.horologist.media.ui.material3.screens.player.PlayerScreen(see https://github.com/google/horologist/pull/2643/changes#diff-cb90d43fff947bc2666a7acd3a67c3993ca017d57b83a04409fbaaa4f8c1898cR36). This is quite a large PR, so I agree to make this step by step for the M3 update

@yschimke
Copy link
Copy Markdown
Collaborator Author

@kul3r4
Copy link
Copy Markdown
Collaborator

kul3r4 commented Apr 24, 2026

@yschimke yschimke marked this pull request as draft May 8, 2026 13:14
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