Skip to content

fix left padding on new details screen styles to fit sidebar - #278

Merged
RadicalMuffinMan merged 2 commits into
Moonfin-Client:masterfrom
jmawet:details-screen-sidebar
Sep 24, 2026
Merged

RadicalMuffinMan merged 2 commits into
Moonfin-Client:masterfrom
jmawet:details-screen-sidebar

Conversation

@jmawet

@jmawet jmawet commented Sep 20, 2026

Copy link
Copy Markdown
Collaborator

Pull Request

Summary

Provide a brief description of what this PR changes and why.

Related Issues

Link related issues or tickets separated by commas.

  • Closes #
  • Fixes #
  • Related to #

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.

  • changed all details screen styles' left padding to 96 to match modern/classic and avoid overlapping the sidebar

Testing

Describe how this change was tested.

  • Tested on physical Roku device
  • Tested via sideload
  • Manual testing completed
  • Not tested (explain why):

Test Steps

Screenshots (if applicable)

  • Nouveau

Before
series-nouveau

After
image

  • Spotlight

Before
series-spotlight

After
image

  • Minimalist

Before
series-minimalist

After
image

Checklist

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

@github-actions

github-actions Bot commented Sep 20, 2026 •

Copy link
Copy Markdown

Build Successful

The channel compiled and packaged. The sideload zip is on the run below.

Property Value
Build Passed
Commit 29a20e3
Workflow run Build #60

… to the new 96px gutters and keep the Spotlight card band on its original 40px bottom gap
@RadicalMuffinMan
RadicalMuffinMan merged commit 698af2c into Moonfin-Client:master Sep 24, 2026
2 checks passed
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.

2 participants