8000
Skip to content

TTML1 3ED Proposed Recommendation - #367

Merged
palemieux merged 7 commits into
masterfrom
issue-0366-create-pr
Oct 4, 2018
Merged

TTML1 3ED Proposed Recommendation#367
palemieux merged 7 commits into
masterfrom
issue-0366-create-pr

Conversation

@nigelmegitt
@nigelmegitt nigelmegitt commented Sep 12, 2018 •
edited
Loading
Copy link
Copy Markdown
Contributor

DO NOT MERGE UNTIL PUBLISHED

Fixes #366. Initial pull request SOTD should be considered "draft" and reviewed.

Build at https://rawgit.com/w3c/ttml1/issue-0366-create-pr-build/index.html

@nigelmegitt nigelmegitt added this to the 3ED PR milestone Sep 12, 2018
@nigelmegitt
nigelmegitt commented Sep 12, 2018
Copy link
Copy Markdown
Contributor Author

Pubrules points out, correctly:

Could not find statements nor links about preliminary implementation/interoperability reports.

So that needs to be added.

@nigelmegitt
nigelmegitt commented Sep 12, 2018 •
edited
Loading
Copy link
Copy Markdown
Contributor Author

Now passes pubrules with warnings:

  • No security and privacy considerations section found. (there is one)
  • The main page should be called Overview.html (instead of index.html). (rename before uploading to /TR)
  • Please verify using Link Checker. done, see TTML1 3ED Proposed Recommendation #367 (comment) for results
  • HTML and CSS validations for compound documents were skipped. Meh

@nigelmegitt
nigelmegitt commented Sep 12, 2018
Copy link
Copy Markdown
Contributor Author

Running Link Checker:

@nigelmegitt
Fix GPS link to use https
c9e1f96 
Half addresses one of the link checker errors (fixing the certificate failure on the site is beyond our control).
@palemieux
palemieux commented Sep 12, 2018
Copy link
Copy Markdown
Contributor

No CSS errors

@palemieux palemieux changed the title Switch to PR entities and SOTD TTML1 3ED Proposed Recommendation Sep 13, 2018
@palemieux
palemieux merged commit 98b8f44 into master Oct 4, 2018
@palemieux
palemieux deleted the issue-0366-create-pr branch October 4, 2018 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@palemieux palemieux palemieux approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

3ED PR

Development

Successfully merging this pull request may close these issues.

2 participants

@nigelmegitt @palemieux
0