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 progress bar calculation #4483

Merged
merged 1 commit into from
Feb 26, 2025

Conversation

lukas-fichtner
Copy link
Contributor

Hello there! This is my first Jellyfin PR – Feedback Welcome!

Changes

  • Corrected condition to properly check if the current time is within the program's duration.
  • Fixed progress percentage calculation using double division for accuracy.
  • Replaced getPremiereDate() with getStartDate() for correct program start time.
  • Refactored getRealTimeProgress() to ensure proper duration calculation.

Issues
Both Live TV progress bars are not working properly.
Couldn't find a matching issue, but this likely affects many. If one exists, please link it. Thanks!

Copy link
Member

@nielsvanvelzen nielsvanvelzen left a comment

Choose a reason for hiding this comment

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

Thanks! Seems like some of these slipped through in a migration I did some time ago 😅

@nielsvanvelzen nielsvanvelzen added this to the v0.19.0 milestone Feb 26, 2025
@nielsvanvelzen nielsvanvelzen added bug Something isn't working live-tv Issue related to the live TV feature backportable Change may be backported to a point release (remove label once cherrypicked) labels Feb 26, 2025
@nielsvanvelzen nielsvanvelzen enabled auto-merge (rebase) February 26, 2025 18:19
@nielsvanvelzen nielsvanvelzen merged commit 2a49b8b into jellyfin:master Feb 26, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backportable Change may be backported to a point release (remove label once cherrypicked) bug Something isn't working live-tv Issue related to the live TV feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants