Skip to content

fix-ui - Spoilermode Extension, the Roku Port - #236

Open
mattsigal wants to merge 1 commit into
Moonfin-Client:masterfrom
mattsigal:feature/extend-spoiler-hiding
Open

fix-ui - Spoilermode Extension, the Roku Port#236
mattsigal wants to merge 1 commit into
Moonfin-Client:masterfrom
mattsigal:feature/extend-spoiler-hiding

Conversation

@mattsigal

Copy link
Copy Markdown
Collaborator

Pull Request

Summary

Extend "Hide Media Description on Details Page" (ui.itemdetail.hideMediaDescription) preference to episode list items and season episode rows on Roku.

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.

  • Updated TVEpisodeListItem.bs to check ui.itemdetail.hideMediaDescription before setting episode overview text.
  • Updated SeasonEpisodeRow.bs to check ui.itemdetail.hideMediaDescription before setting episode overview text.

Testing

Describe how this change was tested.

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

Test Steps

  1. Toggle "Hide Media Description on Details Page" (ui.itemdetail.hideMediaDescription) under Settings -> Personalization.
  2. Verify that episode overviews in TVEpisodeListItem and SeasonEpisodeRow are hidden cleanly when enabled.

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

…items and season rows

Extend spoiler hiding setting to TVEpisodeListItem and SeasonEpisodeRow so episode overviews are hidden when enabled.
@jmawet

jmawet commented Aug 23, 2026

Copy link
Copy Markdown
Collaborator

toggle off:
image

toggle on:
image

image

@mattsigal

Copy link
Copy Markdown
Collaborator Author

Heck yeah, but wow that line spacing is something lol

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