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

[BP] Fix a few markup errors to please pubrules #879

Merged
merged 1 commit into from
May 10, 2017

Conversation

tidoust
Copy link
Member

@tidoust tidoust commented May 10, 2017

Also adjusted encoding issues for some author names (Unicode Normalization Form C should be used), and updated links that yield a 301 permanent redirect to avoid the redirect.

Also adjusted encoding issues for some author names (Unicode Normalization
Form C should be used), and updated links that yield a 301 permanent redirect
to avoid the redirect.
@tidoust tidoust requested a review from 6a6d74 May 10, 2017 15:31
@cportele cportele added the bp label May 10, 2017
@6a6d74 6a6d74 merged commit 54baf72 into w3c:gh-pages May 10, 2017
@andrea-perego
Copy link
Contributor

@tidoust , I think there are still some HTML validation issues - see:

https://validator.w3.org/nu/?doc=http%3A%2F%2Fw3c.github.io%2Fsdw%2Fbp%2Findex.html

@tidoust
Copy link
Member Author

tidoust commented May 10, 2017

@andrea-perego Right but note you're testing the source spec, here, not the generated spec. The errors disappear with the generated spec, except the empty which I had indeed missed and which I had to fix when I prepared the document to publish. I'll create another PR.

@andrea-perego
Copy link
Contributor

Thanks, @tidoust . Yes, I was not actually referring to the warnings.

BTW, please note that, besides the empty <tr>, there are two other errors:

Error: Duplicate attribute class.
At line 4350, column 53

Error: Non-space character in page trailer.
From line 4587, column 1; to line 4587, column 8

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

4 participants