Skip to content

fix home row padding settings - #207

Open
jmawet wants to merge 1 commit into
Moonfin-Client:masterfrom
jmawet:home-row-padding
Open

fix home row padding settings#207
jmawet wants to merge 1 commit into
Moonfin-Client:masterfrom
jmawet:home-row-padding

Conversation

@jmawet

@jmawet jmawet commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

Pull Request

Summary

Fixes the home row padding settings to scale more consistently with core.

Before this the modern row padding options were all way too high, and the classic row padding options scaled too small to where they barely made a difference

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.

  • move home row padding outside of home row toggles to match their location in core
  • scale padding value from flutter default to something that works better visually on roku
  • remove legacy paddingStr as the value will only ever be a number

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)

Classic minimum (10):
image

Classic default (30):
image

Classic medium (80):
image

Classic maximum (130):
image

Modern minimum (360):
image

Modern default (460):
image

Modern maximum (560):
image

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.

1 participant