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(subtitle): fix subtitle position by updating libpgs to v0.8.0 #2487

Closed
wants to merge 1 commit into from

Conversation

Arcus92
Copy link
Contributor

@Arcus92 Arcus92 commented Nov 5, 2024

Changes
This PR updates libpgs-js to version v0.8.0:

  • Fixed a subtitle position issue. The subtitle position now matches the position in FFmpeg.
  • Subtitle cropping is now supported. This is based on the FFmpeg implementation, but I couldn't test it on a real subtitle track yet.

Issues
Fixes jellyfin/jellyfin-web#6273

Related PRs

@jellyfin-bot
Copy link

jellyfin-bot commented Nov 5, 2024

Cloudflare Pages deployment

Latest commit d36c046
Status ✅ Deployed!
Preview URL https://09c3bc85.jf-vue.pages.dev
Preview alias https://update-pgs.jf-vue.pages.dev
Type 🔀 Preview

View build logs

Copy link

sonarcloud bot commented Nov 5, 2024

@Arcus92 Arcus92 changed the title Update dependency libpgs to v0.8.0 fix(subtitle): fix subtitle position by updating libpgs to v0.8.0 Nov 5, 2024
@ferferga
Copy link
Member

ferferga commented Nov 5, 2024

@Arcus92 Thank you for your PR, but this is not needed (unlike the jellyfin-web PR, where further changes were required).
We have dependency updates automated and this is already tracked at #2483

The @jellyfin/sdk update needs some changes in the code and that's why I haven't merged that PR rightaway!

Thank you very much for keeping a track of Jellyfin Vue as well as Web! I highly appreciate it! 🙏

@ferferga ferferga closed this Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

10.10.0 Experimental PGS subtitle render alignment
3 participants