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.

Sign up for GitHub

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

Jump to bottom

IMSC Reader should report out of order <set> elements #60

Closed
palemieux opened this issue Oct 20, 2020 · 0 comments · Fixed by #171
Closed

IMSC Reader should report out of order <set> elements #60

palemieux opened this issue Oct 20, 2020 · 0 comments · Fixed by #171
Assignees
@palemieux
Labels
bug Something isn't working c-imsc-reader for-review proposed resolution is ready for review
Milestone
Milestone 5

Comments

@palemieux
Copy link
Contributor 

<p>
  <set/>
  <span/>
  <set/>
  ...
</p>

should be reported as invalid and the second <set> ignored.
@palemieux palemieux self-assigned this Oct 20, 2020
@palemieux palemieux added this to the Milestone 5 milestone Dec 7, 2020
palemieux added a commit that referenced this issue Dec 14, 2020
@palemieux
Report out-of-order set elements (#60)
f788f9d
@palemieux palemieux added bug Something isn't working for-review proposed resolution is ready for review labels Dec 14, 2020
@palemieux palemieux mentioned this issue Dec 14, 2020
Merged
palemieux added a commit that referenced this issue Dec 15, 2020
@palemieux
Report out-of-order <set> elements (#60)
98b28ec
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@palemieux palemieux

Labels
bug Something isn't working c-imsc-reader for-review proposed resolution is ready for review
Projects
None yet
Milestone
Milestone 5
Development

Successfully merging a pull request may close this issue.

Report out-of-order set elements sandflow/ttconv
1 participant
@palemieux