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

fix dead link #7199

Closed
wants to merge 3 commits into from
Closed

fix dead link #7199

wants to merge 3 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Oct 11, 2021

The htmlwg link is dead so replaced with working link from archive

BTW, some of the MDN links are also dead (moved to a different location on the same mozilla site). but i couldn't see those mdn links in the source, so is that a problem for this or for
https://github.com/w3c/mdn-spec-links ?


/browsing-the-web.html ( diff )
/infrastructure.html ( diff )
/origin.html ( diff )
/references.html ( diff )

@annevk
Copy link
Member

annevk commented Oct 12, 2021

@mahanstreamer thanks, but can you update these with https://datatracker.ietf.org/doc/html/rfc8941 instead?

And which MDN links are broken? @sideshowbarker can probably help with giving pointers for fixing those.

@ghost
Copy link
Author

ghost commented Oct 13, 2021

The dead MDN link is developer.mozilla.org/en-US/docs/Web/HTML/Introduction_to_HTML/Document_and_website_structure and https://developer.mozilla.org/en-US/docs/Web/Forms/Form_validation

Comment on lines +2432 to +2434
<li><dfn data-x="http-structured-header-boolean" data-x-href="https://datatracker.ietf.org/doc/html/rfc8941">boolean</dfn></li>
<li><dfn data-x="http-structured-header-token" data-x-href="https://datatracker.ietf.org/doc/html/rfc8941">token</dfn></li>
<li><dfn data-x="http-structured-header-parameters" data-x-href="https://datatracker.ietf.org/doc/html/rfc8941">parameters</dfn></li>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<li><dfn data-x="http-structured-header-boolean" data-x-href="https://datatracker.ietf.org/doc/html/rfc8941">boolean</dfn></li>
<li><dfn data-x="http-structured-header-token" data-x-href="https://datatracker.ietf.org/doc/html/rfc8941">token</dfn></li>
<li><dfn data-x="http-structured-header-parameters" data-x-href="https://datatracker.ietf.org/doc/html/rfc8941">parameters</dfn></li>
<li><dfn data-x="http-structured-header-boolean" data-x-href="https://datatracker.ietf.org/doc/html/rfc8941#section-3.3.6">boolean</dfn></li>
<li><dfn data-x="http-structured-header-token" data-x-href="https://datatracker.ietf.org/doc/html/rfc8941#section-3.3.4">token</dfn></li>
<li><dfn data-x="http-structured-header-parameters" data-x-href="https://datatracker.ietf.org/doc/html/rfc8941#section-3.1.2">parameters</dfn></li>

domenic added a commit that referenced this pull request Feb 17, 2022
domenic added a commit that referenced this pull request Feb 18, 2022
mfreed7 pushed a commit to mfreed7/html that referenced this pull request Jun 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant