Skip to content

feat: Chapter image trickplay/timeline fallback#963

Draft
matt-hb wants to merge 8 commits into
DonutWare:developfrom
matt-hb:chapter-image-fallback
Draft

feat: Chapter image trickplay/timeline fallback#963
matt-hb wants to merge 8 commits into
DonutWare:developfrom
matt-hb:chapter-image-fallback

Conversation

@matt-hb

@matt-hb matt-hb commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

Pull Request Description

First actual feature PR, chapter images have been bothering me a bunch so I'm implementing a trickplay fallback as per #875 .
Would love if someone could try it out on bigger screens on Android TV to see how it looks, since currently I am blurring trickplay images a bit as they are lower resolution (I think a bit of blur is nicer than pixelation).
Also added caching for trickplay images, because we're fetching them a bunch.

Issue Being Fixed

Placeholder when there are no chapter images available, either via fallback to trickplay images or a timeline if trickplay images are not available either.

Screenshots / Recordings

Before (yucky placeholders):
image

After (with trickplay fallback):
image

After (with timeline):
TODO

Tested On

  • Android
  • Android TV
  • Desktop
  • Web

Checklist

  • Check that any changes are related to the issue at hand.
  • Create timeline widget
  • Implement in VideoPlayerChapters?

@matt-hb

matt-hb commented Apr 17, 2026

Copy link
Copy Markdown
Contributor Author

looks like some old commits from another PR are still there, they don't do anything for the diff though, my bad!

@PartyDonut

Copy link
Copy Markdown
Collaborator

Just did a test for Android TV it's quite blurry. Does not look great to be honest.

Another solution would be to change the filterquality inside of the _TrickplayPainter from trick_play_image.dart

image

@matt-hb

matt-hb commented Apr 23, 2026

Copy link
Copy Markdown
Contributor Author

Another solution would be to change the filterquality inside of the _TrickplayPainter from trick_play_image.dart

Good call, I misunderstood low level drawing and size inheriting before, but after checking this is in fact a lot better.

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