Skip to content

Align full player placeholder spacing#560

Merged
theovilardo merged 7 commits intobckp/player-ui-optimfrom
codex/fix-vertical-spacing-for-placeholders
Dec 18, 2025
Merged

Align full player placeholder spacing#560
theovilardo merged 7 commits intobckp/player-ui-optimfrom
codex/fix-vertical-spacing-for-placeholders

Conversation

@theovilardo
Copy link
Copy Markdown
Owner

Summary

  • adjust metadata placeholder layout to mirror real section sizing
  • align progress placeholder padding and sizing with the actual progress bar
  • bring controls placeholder height in line with the live controls stack

Testing

  • Not run (not requested)

Codex Task

Polishes the loading placeholder in the full player and refactors the `FileExplorerBottomSheet` to use a standard `CenterAlignedTopAppBar`.

**`FullPlayerContent.kt`:**
- Adjusts spacing, sizing, and corner radius of various placeholder elements for a more refined look.
- Modifies the placeholder icon's tint color and removes the alpha from the background surface.
- Increases the minimum height and spacing in the metadata and controls sections.

**`FileExplorerBottomSheet.kt`:**
- Replaces the custom header `Row` with a standard `CenterAlignedTopAppBar`.
- This change centralizes the title and moves the close and refresh buttons into the standard `navigationIcon` and `actions` slots.

**Other Changes:**
- **`SettingsScreen.kt`**: Updates the "Experimental" settings icon from a warning symbol to a science beaker.
- **`ExperimentalSettingsScreen.kt`**: Reduces padding and spacing for a more compact layout.
@theovilardo theovilardo merged commit 70080e4 into bckp/player-ui-optim Dec 18, 2025
@lostf1sh lostf1sh deleted the codex/fix-vertical-spacing-for-placeholders branch January 16, 2026 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant