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: Fix rendering of image subs when using SimpleTextDisplayer #7258

Merged

Conversation

avelad
Copy link
Member

@avelad avelad commented Sep 4, 2024

This change prevents cues from being created without payload, as is the case with subtitles in image format.

@avelad avelad added type: bug Something isn't working correctly component: captions/subtitles The issue involves captions or subtitles priority: P2 Smaller impact or easy workaround labels Sep 4, 2024
@avelad avelad added this to the v4.11 milestone Sep 4, 2024
@shaka-bot
Copy link
Collaborator

Incremental code coverage: 100.00%

@avelad avelad marked this pull request as draft September 4, 2024 15:59
@avelad avelad marked this pull request as ready for review September 5, 2024 12:23
@avelad avelad merged commit 3d0b817 into shaka-project:main Sep 5, 2024
34 of 36 checks passed
@avelad avelad deleted the fix-image-subs-simple-text-displayer branch September 5, 2024 13:56
avelad added a commit that referenced this pull request Sep 9, 2024
This change prevents cues from being created without payload, as is the
case with subtitles in image format.
avelad added a commit that referenced this pull request Sep 9, 2024
This change prevents cues from being created without payload, as is the
case with subtitles in image format.
@avelad avelad modified the milestones: v4.11, v4.12 Sep 12, 2024
joeyparrish pushed a commit that referenced this pull request Sep 13, 2024
This change prevents cues from being created without payload, as is the
case with subtitles in image format.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: captions/subtitles The issue involves captions or subtitles priority: P2 Smaller impact or easy workaround type: bug Something isn't working correctly
3 participants