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

SRT subtitles not displayed when "Text Subtitles Only" option is enabled #1627

Closed
dathbe opened this issue Jan 9, 2024 · 4 comments · Fixed by #1633
Closed

SRT subtitles not displayed when "Text Subtitles Only" option is enabled #1627

dathbe opened this issue Jan 9, 2024 · 4 comments · Fixed by #1633
Labels
bug Something isn't working

Comments

@dathbe
Copy link

dathbe commented Jan 9, 2024

Software Versions

  • Jellyfin Server Version: 10.8.13
  • Roku Client Version: 2.0.1 (2.0 build 1)

Describe the bug

When the "Text Subtitles Only" setting is enabled, the app hides SRT subtitles, which are text subtitles.

How To Reproduce

  1. Enable "Text Subtitles Only" (Settings>Playback>Text Subtitles Only)
  2. Play a movie with SRT subtitles
  3. Press the up button to enter the option menu
  4. SRT subtitles are not available

Expected behavior

"Text Subtitles Only" should only hide graphic subtitles (e.g., PGS)

Logs

N/A

Screenshots

compare

Connection Information

  • Is server local or remote? Local
  • Is server connection HTTP or HTTPS? HTTPS

Additional context

@1hitsong
Copy link
Member

1hitsong commented Jan 12, 2024

Found the root cause. Working on a fix.

The code only allows a subtitle track to have 1 type. forced, default, text, or normal. In this case, the subtitle track is both default AND text, but only the default will get picked up.

image

@1hitsong
Copy link
Member

@dathbe Please test this PR and see if it resolves your issue: #1633

@dathbe
Copy link
Author

dathbe commented Jan 12, 2024

Confirmed that PR 1633 fixes this issue for me. Text subtitles are now shown with the "Text Subtitles Only" option enabled, even if they are set to default. Thanks!

@1hitsong
Copy link
Member

Fixed by #1633

cewert pushed a commit that referenced this issue Jan 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants