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

Change Subtitle Sync slider to go from -300 to 300 - Backport #4754 #4893

Merged
merged 1 commit into from
Oct 19, 2023

Conversation

thornbill
Copy link
Member

@thornbill thornbill commented Oct 19, 2023

Changes
Backport the subtitle sync slider range change from #4754 to the 10.8 release branch

Issues
Fixes #4753

This patch changes the subtitle sync from using a procentage to a
'slider value' that ranges from -300 to 300. The reasons for this is that
WebOS doesn't jump in 0.1 increments but instead jumps 1.0 increments in
the slider, which results in subtitle sync jumping 0.6s per increment.
Using a value from -300 to 300 makes LG WebOS jump 0.1s instead.
@thornbill thornbill added bug Something isn't working p: webos This PR or issue mainly concerns WebOS clients ui & ux This PR or issue mainly concerns UI & UX labels Oct 19, 2023
@thornbill thornbill added this to the v10.8.12 milestone Oct 19, 2023
@sonarcloud
Copy link

sonarcloud bot commented Oct 19, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@thornbill thornbill merged commit aa09fa7 into jellyfin:release-10.8.z Oct 19, 2023
14 checks passed
@thornbill thornbill deleted the backport-4754 branch October 19, 2023 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working p: webos This PR or issue mainly concerns WebOS clients ui & ux This PR or issue mainly concerns UI & UX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants