Skip to content

Commit

Permalink
Add step to run accessible HTML through validator, see phetsims/gravi…
Browse files Browse the repository at this point in the history
  • Loading branch information
jessegreenberg committed Jun 28, 2019
1 parent ef698d0 commit c8bebd6
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions checklists/code_review_checklist.md
Original file line number Diff line number Diff line change
Expand Up @@ -550,6 +550,7 @@ Because JavaScript lacks visibility modifiers (public, protected, private), PhET

## **Accessibility**
- [ ] Does the sim pass an accessibility fuzz test? (run with query parameters `fuzzBoard&ea`)
- [ ] Run the accessible HTML through an [HTML validator](https://validator.w3.org/nu/#textarea), does the HTML pass?
- [ ] Are accessibility features integrated well into the code. They should be added in a maintainable way, even if that requires upfront refactoring.
- [ ] Are accessible design patterns used, see [accessible-design-patterns.md](https://github.com/phetsims/phet-info/blob/master/doc/accessible-design-patterns.md)
- [ ] Does resetting the simulation also reset the entire PDOM?
Expand Down

0 comments on commit c8bebd6

Please sign in to comment.