Skip to content

fix: fix breaking captions#3556

Merged
michellezhuogg merged 4 commits intoshaka-project:masterfrom
bcupac:fix_captions
Aug 16, 2021
Merged

fix: fix breaking captions#3556
michellezhuogg merged 4 commits intoshaka-project:masterfrom
bcupac:fix_captions

Conversation

@bcupac
Copy link

@bcupac bcupac commented Jul 28, 2021

Description

Fixes #3555

Screenshots (optional)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to
    not work as expected)
  • This change requires a documentation update

Checklist:

  • I have signed the Google CLA https://cla.developers.google.com
  • My code follows the style guidelines of this project
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have verified my change on multiple browsers on different platforms
  • I have run ./build/all.py and the build passes
  • I have run ./build/test.py and all tests pass

@bcupac bcupac changed the title fix: fix breaking captions when a cue has a forward slash in text fix: fix breaking captions Jul 29, 2021
Copy link
Contributor

@michellezhuogg michellezhuogg left a comment

Choose a reason for hiding this comment

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

Thank you for fixing this!

@shaka-bot
Copy link
Collaborator

All tests passed!

@michellezhuogg michellezhuogg merged commit 65e9d8f into shaka-project:master Aug 16, 2021
joeyparrish pushed a commit that referenced this pull request Oct 12, 2021
joeyparrish pushed a commit that referenced this pull request Oct 12, 2021
@github-actions github-actions bot added the status: archived Archived and locked; will not be updated label Jul 25, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

status: archived Archived and locked; will not be updated

Projects

None yet

Development

Successfully merging this pull request may close these issues.

captions - captions break if the vvt loaded contains a cue that has a forward slash in its text

3 participants