Skip to content

Multi version fixes for episodes#7984

Open
Shadowghost wants to merge 15 commits into
jellyfin:masterfrom
Shadowghost:version-aware-skip
Open

Multi version fixes for episodes#7984
Shadowghost wants to merge 15 commits into
jellyfin:masterfrom
Shadowghost:version-aware-skip

Conversation

@Shadowghost

@Shadowghost Shadowghost commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

When watching version "Colored" of an item in a playlist or a season, the next played item will always be the main version of the next item. We should use the version that matches the currently played if possible.

Changes

  • When skipping, select the version that name-matches the currently played one
  • When switching versions in the selector, reload item data:
    • Fixes shown chapters not being aligned with the selected version

Issues

Code assistance

Claude Opus for debugging support


  • I have read and followed the contributing guidelines.
  • I have tested these changes.
  • I have verified that this is not duplicating changes in an existing PR.
  • I have provided a substantive review of another web PR.

@Shadowghost
Shadowghost requested a review from a team as a code owner June 2, 2026 08:39
@jellyfin-bot

jellyfin-bot commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

Cloudflare Pages deployment

Latest commit f5c5dc4
Status ✅ Deployed!
Preview URL https://7b5de56a.jellyfin-web.pages.dev
Type 🔀 Preview

View build logs

Comment thread src/components/playback/playbackmanager.js
Comment thread src/components/playback/playbackmanager.js Outdated
Comment thread src/components/playback/playbackmanager.js
@Shadowghost
Shadowghost force-pushed the version-aware-skip branch from 56e769d to ff7e2ae Compare June 3, 2026 09:49

@dfederm dfederm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM! (Not that my approval is official)

@Shadowghost Shadowghost changed the title Select version of next played item based on current Multi version fixes for episodes Jun 5, 2026
@theguymadmax theguymadmax removed the status in Jellyfin 12.0 Jun 7, 2026
@theguymadmax theguymadmax moved this to Todo in Jellyfin 12.0 Jun 7, 2026
@thornbill thornbill moved this from Todo to In Progress in Jellyfin 12.0 Jun 12, 2026
@thornbill thornbill moved this from In Progress to Review in Jellyfin 12.0 Jun 13, 2026
@thornbill thornbill added the bug Something isn't working label Jun 13, 2026
@thornbill thornbill added this to the v12.0 milestone Jun 13, 2026
@thornbill thornbill moved this from Review to In Progress in Jellyfin 12.0 Jun 13, 2026
@jellyfin-bot jellyfin-bot added the merge conflict Conflicts prevent merging label Jun 30, 2026
@jellyfin-bot

Copy link
Copy Markdown
Collaborator

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

@jellyfin-bot jellyfin-bot added the stale No activity for an extended length of time label Jul 1, 2026
@jellyfin-bot jellyfin-bot removed the merge conflict Conflicts prevent merging label Jul 1, 2026
@thornbill thornbill removed the stale No activity for an extended length of time label Jul 1, 2026
@jellyfin-bot jellyfin-bot added the merge conflict Conflicts prevent merging label Jul 8, 2026
@jellyfin-bot

Copy link
Copy Markdown
Collaborator

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

@jellyfin-bot jellyfin-bot added the stale No activity for an extended length of time label Jul 9, 2026
@jellyfin-bot jellyfin-bot removed the merge conflict Conflicts prevent merging label Jul 13, 2026
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working stale No activity for an extended length of time

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

5 participants