Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Trickplay thumbnail in older web engines #5719

Merged
merged 1 commit into from
Jul 15, 2024

Conversation

dmitrylyzo
Copy link
Contributor

Changes
Replace image with CSS background.

Issues
In webOS 1.2, the image is scaled to fit in the chapterThumpWrapper and ignores position. Since it doesn't seem to support resize nor object-fit to prevent scaling, use background instead.

I avoided reusing chapterThumb with necessary CSS changes since it is used by chapters.

In webOS 1.2, the image is scaled to fit in the `chapterThumpWrapper`
and ignores position. Since it doesn't seem to support `resize` nor
`object-fit` to prevent scaling, use background instead.
@dmitrylyzo dmitrylyzo added bug Something isn't working ui & ux This PR or issue mainly concerns UI & UX needs testing This PR requires additional testing labels Jun 18, 2024
@dmitrylyzo dmitrylyzo marked this pull request as ready for review June 20, 2024 10:05
@dmitrylyzo dmitrylyzo requested a review from a team as a code owner June 20, 2024 10:05
@dmitrylyzo dmitrylyzo changed the title Fix Trickplay thumbnail Fix Trickplay thumbnail in older web engines Jun 20, 2024
@thornbill thornbill added the stable backport Backport into the next stable release label Jul 12, 2024
@thornbill thornbill added this to the v10.9.8 milestone Jul 12, 2024
Copy link
Member

@thornbill thornbill left a comment

Choose a reason for hiding this comment

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

Works for me in (current) Firefox.

@thornbill thornbill removed the needs testing This PR requires additional testing label Jul 15, 2024
@thornbill thornbill merged commit 9501c50 into jellyfin:release-10.9.z Jul 15, 2024
8 checks passed
@dmitrylyzo dmitrylyzo deleted the fix-bubble-trickplay branch July 16, 2024 15:11
@jellyfin-bot jellyfin-bot removed the stable backport Backport into the next stable release label Jul 21, 2024
joshuaboniface pushed a commit that referenced this pull request Jul 21, 2024
Fix Trickplay thumbnail in older web engines

Original-merge: 9501c50

Merged-by: thornbill <thornbill@users.noreply.github.com>

Backported-by: Bill Thornton <thornbill@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ui & ux This PR or issue mainly concerns UI & UX
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants