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

Broken links and fragments #28

Closed
vivienlacourba opened this issue Mar 18, 2016 · 3 comments
Closed

Broken links and fragments #28

vivienlacourba opened this issue Mar 18, 2016 · 3 comments
Assignees

Comments

@vivienlacourba
Copy link
Member

Result of: https://validator.w3.org/checklink?url=https://w3c.github.io/webrtc-stats/webrtc-stats.html

 Line: 186 https://w3c.github.io/webrtc-stats/webrtc-stats.html
    Status: 200 OK

    Some of the links to this resource point to broken URI fragments (such as index.html#fragment). 
    Broken fragments:

        https://w3c.github.io/webrtc-stats/webrtc-stats.html#widl-RTCCodec-codec (line 186)

warning Lines: 70, 77, 80, 84 http://dev.w3.org/html5/spec/webappapis.html redirected to http://w3c.github.io/html/webappapis.html
    Status: 301 -> 200 OK

    This is a permanent redirect. The link should be updated to point to the more recent URI. 
    Broken fragments:

        http://dev.w3.org/html5/spec/webappapis.html#event-handlers (line 84)
        http://dev.w3.org/html5/spec/webappapis.html#fire-a-simple-event (line 80)
        http://dev.w3.org/html5/spec/webappapis.html#queue-a-task (line 77)
        http://dev.w3.org/html5/spec/webappapis.html#eventhandler (line 70)
@vivienlacourba
Copy link
Member Author

http://dev.w3.org/html5/spec/webappapis.html is now a redirect to https://w3c.github.io/html/webappapis.html which is section "7. Web application APIs" of the HTML 5.1 Editor's Draft.

I believe our dependency is on HTML5 not HTML 5.1. If true then we should link to the HTML5 REC https://www.w3.org/TR/2014/REC-html5-20141028/webappapis.html

Also Eventhandler is a callback function not an interface.

@vr000m
Copy link
Contributor

vr000m commented May 23, 2016

@alvestrand
Copy link
Contributor

@vivienlacourba can you look again and verify that it's fixed? Closing issue for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants