Skip to content

feat(tv): Add video seek buttons on the native player - #73

Open
xayron wants to merge 1 commit into
Spyou:mainfrom
xayron:feature/tv-player-seek-buttons
Open

feat(tv): Add video seek buttons on the native player#73
xayron wants to merge 1 commit into
Spyou:mainfrom
xayron:feature/tv-player-seek-buttons

Conversation

@xayron

@xayron xayron commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

What changed

Added seek buttons (forward and backward) to native TV player, and added preference to turn it off and select custom duration of seek

Closes #

How you tested it

Checklist

  • flutter analyze is clean
  • flutter test passes
  • Native build still compiles, if I touched android/ or ios/
  • I've read and agree to the CLA
  • New dependency or third-party code? NOTICE.md updated in this PR
  • Used AI tooling? Disclosed per AI_POLICY.md

Screenshots

image (2) image (1) image

@NeighborhoodNerd

Copy link
Copy Markdown
Collaborator

Nice addition — the TV seek buttons look useful.

One thing to fix before merge: seek duration should be split into separate prefs for mobile vs TV. Right now TV seek buttons and mobile double-tap skip both use seekSeconds, and the new "Seek Duration" setting can show on phone alongside "Double-tap skip" even though they edit the same value. A dedicated TV seek duration (and keeping double-tap on its own mobile setting) would avoid cross-platform coupling and the duplicate settings row on phone.

@NeighborhoodNerd
NeighborhoodNerd self-requested a review August 31, 2026 21:07

@NeighborhoodNerd NeighborhoodNerd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Split the seekDuration for TV and double-tap seek settings for mobile.

Comment on lines +128 to +129
/// [seekBuuttons] enables the buttons; [seekSeconds] is the jump size, clamped 5–180s (default 10).
bool get seekButtons => _box.get('seekButtons', defaultValue: true) as bool;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix typo. seekBuuttons -> seekButtons

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