Skip to content

UI: Add 8.0 top padding to library browse screen header#410

Merged
RadicalMuffinMan merged 1 commit into
Moonfin-Client:mainfrom
mattsigal:feature/home-movies-padding
Jun 7, 2026
Merged

UI: Add 8.0 top padding to library browse screen header#410
RadicalMuffinMan merged 1 commit into
Moonfin-Client:mainfrom
mattsigal:feature/home-movies-padding

Conversation

@mattsigal

@mattsigal mattsigal commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Pull Request

Summary

This aligns the title text height on library browse screens (such as Home Movies and Playlists) with the music/audio browse screen header, resolving the issue where the library header was placed too high on TV and mobile layouts.

Add 8.0 top padding to library browse screen header to align it with the music/audio browse screen header, resolving the issue where the library header was placed too high on TV and mobile layouts.

Related Issues

Link related issues or tickets separated by commas.

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Performance improvement
  • UI/UX update
  • Documentation update
  • Build/CI change
  • Other (describe):

Changes Made

List the key changes included in this PR.

  • Added a + 8.0 padding offset to topPad in _LibraryHeader inside library_browse_screen.dart to match _MusicHeader padding behavior.

Platform

  • Android
  • iOS
  • macOS
  • Windows
  • Linux
  • All / Shared code

Testing

  • Tested on emulator / simulator
  • Tested on physical device
  • Manual testing completed
  • Not tested (explain why):
    • Code analyzed and verified with flutter analyze (no issues found).
    • Matches padding behavior of _MusicHeader which has been visually verified.
    • I'm tired and haven't had coffee yet

Test Steps

  1. Navigate to a library page (Home Movies, Playlists, Movies, etc.) and check title height.
  2. Verify alignment with the Audio page title.

Screenshots (if applicable)

Include screenshots or recordings for UI changes.

Checklist

  • Code builds successfully
  • Code follows project style and conventions
  • No unnecessary commented-out code
  • No new warnings introduced

This aligns the title text height on library browse screens (such as Home Movies and Playlists) with the music/audio browse screen header, resolving the issue where the library header was placed too high on TV and mobile layouts.
@RadicalMuffinMan RadicalMuffinMan merged commit f11be91 into Moonfin-Client:main Jun 7, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[UI] In the "Home Videos and Photos" category, alongside movies and series, the top half is cut off.

2 participants