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

Addresses #163 (Background re forcedDisplay) #170

Merged
merged 9 commits into from
Feb 12, 2016
Merged

Addresses #163 (Background re forcedDisplay) #170

merged 9 commits into from
Feb 12, 2016

Conversation

palemieux
Copy link
Contributor

Closes #163

palemieux and others added 9 commits February 4, 2016 10:52
* Change EBU-TT-D encoding statement to remove specific reference to
UTF-16 and mention EBU-TT-D recommendation for UTF-8.
* Make EBU-TT-D example validate against EBU-TT-D schema by moving
`tts:lineHeight` specification from `<body>` to `"baseStyle" <style>`.
* Fix typo of supplemented
* Tighten up wording re SMPTE-TT constraints and remove redundant
bullet.
This commit adds the complete example files from the spec, so that they
can be individually loaded and validated. This doesn’t help with
synchronising changes between the HTML document and the examples of
course, so it’s just a handy reference material. Arguably they should
have the .ttml extension since that’s the registered type extension,
but xml is not exactly wrong and is more easily processed by generic
XML processors.
Address line comments for issue-126 branch made by @nigelmegitt
Made string literals consistent in forcedDisplay section
@palemieux palemieux self-assigned this Feb 11, 2016
@palemieux palemieux modified the milestones: imsc1-cr4, imsc1-pr Feb 11, 2016
@nigelmegitt
Copy link
Contributor

LGTM

palemieux added a commit that referenced this pull request Feb 12, 2016
Addresses #163 (Background re forcedDisplay)
@palemieux palemieux merged commit d6287cb into master Feb 12, 2016
@palemieux palemieux deleted the issue-163 branch March 10, 2016 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants