Skip to content

homeRows and detailsSection style changes#116

Open
harleykradovill wants to merge 5 commits into
Moonfin-Client:masterfrom
harleykradovill:detailssection
Open

homeRows and detailsSection style changes#116
harleykradovill wants to merge 5 commits into
Moonfin-Client:masterfrom
harleykradovill:detailssection

Conversation

@harleykradovill

Copy link
Copy Markdown
Collaborator

Pull Request

Summary

This PR updates the styling of homeRows and the detailsSection above them. It adds rounded corners to posters, replaces the focus image, and limits the focus region to the poster itself. The detailsSection text now matches the sizing and spacing of the media bar.

Type of Change

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

Changes Made

  • Replaced the focus image for home items. Confirmed that the 9-patch works correctly for standard, wide, and music posters.
  • Changed HomeRowItemsSizes so the focus cursor only appears around the poster, just like the core app.
  • Added poster masks for poster, wide, and music. This also caused alignment issues with the progress bar and played indicator. The progress bar is now less wide than the poster, and translated upward. The played indicator was moved slightly inward.
  • Moved the translation of item title closer to the poster, and changed the font.
  • Changed the font sizes and spacing of the detailsSection to match the media bar.

Testing

This was tested using the demo server for Jellyfin. I confirmed that Movies, TV Shows, and Music all display their masks correctly. Posters that do not match these sizes will not display a mask. I also made sure the TMDB/MDBList ratings displayed correctly by testing with my own server.

Further testing with Kefintweaks and HSS may be needed. Nothing in this PR should break those, but styling wise I am not sure.

Screenshots

Before:
screenshot-2026-06-17-01 12 34 334

After:
screenshot-2026-06-17-01 00 19 723

Checklist

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

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