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

Accessibility test standardization, improvements and documentation #83

Closed
4 tasks done
rahimabdi opened this issue Nov 16, 2023 · 14 comments
Closed
4 tasks done

Comments

@rahimabdi
Copy link

rahimabdi commented Nov 16, 2023

Tracker:


@cookiecrook Consolidating general thoughts here on improving the quality/consistency of our WPT accessibility test files and providing supporting documentation for test authors (also to make it less daunting for WPT contributors). All feedback welcome! Happy to take this forward after you and other folks have provided input.

I know that detailed guidance is already captured here and this may be moved to web-platform-tests/interop-accessibility: Running the WPT Tests for ARIA and Related Specs.

  • Writing tests, guidance and implementation practices
    • Boilerplate comment header for all test files (e.g., "This file tests X...")
    • Providing documentation links to relevant parts of a spec being tested
    • Comment links to where relevant testing is located (and documentation scheme for referencing files in other locations)
    • Naming .html test files (usually after a spec's section)
    • Readability improvement guidance: e.g., alphabetization where possible, prescriptive order of element attributes (e.g., <div id="id" data-expectedtestname="someTest" data-expectedlabel="label">, spacing and indentation, etc.
    • Subtest names must be globally unique (the more descriptive the better)
    • id naming scheme for elements/containers (e.g., many elements with unique ids referenced via aria-labelledby)
    • Performing HTML validation
    • Using PR checks (primarily ensuring that lint is clean, removing whitespace)
    • PR commit message (description and co-author attribution)
  • Supporting documentation for test authors/maintainers
    • Overview of the WPT project
    • Understanding/viewing CI results
    • More on testharness.js
    • Index page with all a11y related tests (e.g., accname, wai-aria, core-arm, html-aam, svg-aam, etc.)
    • WPT Debugging Q&A and common issues (e.g., troubleshooting bad data* attribute on test, non-unique test name, improper use of aria-utils utility functions)
    • Explainer for aria-utils.js and what it does
@rahimabdi rahimabdi self-assigned this Nov 16, 2023
@cookiecrook
Copy link
Collaborator

Generally all sounds good. I think some of the non-accessibility-specific stuff you mentioned above may already be documented in the main WPT project... Any accessibility-specific docs could go here, along with pointers to the main WPT docs where relevant.

@boazsender
Copy link
Contributor

boazsender commented Jan 31, 2024

@rahimabdi @lolaodelola and I met and discussed a few additional ideas:

  • We talked about creating a guide for where to put tests
  • Create an index page of all the tests using the tags
  • Going back and tagging existing tests
  • Linking to these docs from web-platform-tests.org

We also talked about our multiple audiences

  • People talking about different accessibility tests (e.g. aria test writers)
  • people in other groups who want to test accessibility

@cookiecrook
Copy link
Collaborator

Adding Agenda+ for docs discussion in the May meeting.

@rahimabdi
Copy link
Author

@boazsender @cookiecrook I've emailed you (and Lola) a link to the draft documentation I'm proposing. Your thoughts and feedback are welcome!

I've also added a tracker to the issue description with outstanding tasks/questions.

@rahimabdi
Copy link
Author

@cookiecrook
Copy link
Collaborator

@chrishtr you took an action to review. Thank you!

@foolip
Copy link
Member

foolip commented Jul 2, 2024

Determine publication platform (likely our GitHub Wiki page)

@chrishtr asked me, and I think most of that document would make sense in https://github.com/web-platform-tests/wpt/tree/master/docs/writing-tests, as a new accessibility.md file most likely. It would then be published to https://web-platform-tests.org/.

Parts of the document are about the Accessibility Interop Group, and that I'd suggest putting right here in this repo's README where it's easy to find or stumble upon.

Some of the FAQ questions could probably be put into the main WPT docs and some others are about the Accessibility Interop Group. Splitting that and the overall doc would be shame in a way since it's useful as a rather self-contained resource. An option could to publish it as a whole in this repo as well, but just note which parts are also available on https://web-platform-tests.org/ and which is the source of truth.

Hope that helps.

@rahimabdi
Copy link
Author

rahimabdi commented Jul 14, 2024

Thanks for chiming in @foolip.

Splitting that and the overall doc would be shame in a way since it's useful as a rather self-contained resource. An option could to publish it as a whole in this repo as well, but just note which parts are also available on https://web-platform-tests.org/ and which is the source of truth.

Agreed, I would also vote for keeping it self-contained in this repo but could be split up otherwise.

@jcsteh
Copy link

jcsteh commented Sep 2, 2024

Thanks for all the work that has gone into this. Is feedback still welcome on the Google Docs draft? If so, could I have comment access? Or is there somewhere else I should provide feedback? I only had some minor clarifications in mind, nothing major. Thanks.

@rahimabdi
Copy link
Author

@jcsteh I've just added you: Accessibility Interop Project Documentation DRAFT.

Thank you for the review/feedback!

@rahimabdi
Copy link
Author

I've published the documentation to the Accessibility Interop's Wiki: https://github.com/web-platform-tests/interop-accessibility/wiki/Accessibility-Interop-Project-Overview-and-Contribution-Guidelines.

Many thanks @gr3g-apple-com @jcsteh for taking the time to review.

@zcorpan @cookiecrook I'll be unavailable for the Oct 1 2024 meeting but could we please add this to the agenda. Unless there is any remaining feedback, this issue is nearing completion.

@cookiecrook
Copy link
Collaborator

cookiecrook commented Oct 1, 2024

@cookiecrook
Copy link
Collaborator

cookiecrook commented Oct 1, 2024

Okay to close now that the follow-ons are tracked separately. Thanks @rahimabdi!

@rahimabdi
Copy link
Author

rahimabdi commented Oct 6, 2024

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

5 participants