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 broken links in HTML accessibility API Mappings 1.0 (#447) #555

Open
wants to merge 2 commits into
base: gh-pages
Choose a base branch
from

Conversation

arbaz686
Copy link

@arbaz686 arbaz686 commented Aug 2, 2024

This update addresses and fixes broken references found in the HTML Accessibility API Mappings 1.0 specification. Specifically, the following changes have been made:

  • Updated URL for param attribute: Replaced the incorrect link to the param attribute documentation with the correct one.
  • Fixed URL for itemprop attribute: Corrected the link pointing to the itemprop attribute section in the microdata specification.
  • Corrected URL for sectioning-root: Updated the link to the correct section in the HTML specification for sectioning root elements.

Details:

  • Fixed Links:
    • https://html.spec.whatwg.org/multipage/iframe-embed-object.html#attr-param-name
    • https://html.spec.whatwg.org/multipage/microdata.html#attr-itemprop
    • https://html.spec.whatwg.org/multipage/sections.html#sectioning-root

Implementation:

  • WPT Tests: No new WPT tests were added as the changes are related to link corrections.
  • Implementations: N/A
    • WebKit: Not applicable
    • Gecko: Not applicable
    • Blink: Not applicable

Closes: #447


💥 Error: 500 Internal Server Error 💥

PR Preview failed to build. (Last tried on Aug 2, 2024, 6:57 AM UTC).

More

PR Preview relies on a number of web services to run. There seems to be an issue with the following one:

🚨 Spec Generator - Spec Generator is the web service used to build specs that rely on ReSpec.

🔗 [Related URL]([object Object])

🕵️‍♀️  That doesn't seem to be a ReSpec document. Please check manually: http://localhost:8082/uploads/njxKbI/index.html?isPreview=true%3FisPreview%3Dtrue&publishDate=2024-08-02

If you don't have enough information above to solve the error by yourself (or to understand to which web service the error is related to, if any), please file an issue.

@scottaohara
Copy link
Member

thanks for finding these. the spec is no longer being updated in this repo. so i will make sure these updates are made in the new location of the document.

@arbaz686
Copy link
Author

arbaz686 commented Aug 2, 2024

could you please share the details or the link to the new repository where the document is being maintained? i would like to ensure that the necessary updates are made there

@pkra
Copy link
Member

pkra commented Aug 3, 2024

@rahimabdi
Copy link
Contributor

rahimabdi commented Aug 7, 2024

Thank you @arbaz686! For clarification, if you'd like to submit the changes, please raise this PR against the ARIA spec.

Link to the new location for the html-aam specification: https://github.com/w3c/aria/tree/main/html-aam.

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

Successfully merging this pull request may close these issues.

4 participants