From 282afaa6a21a3b888a7a69a3e1ffaf9304600c21 Mon Sep 17 00:00:00 2001 From: Mai Irie Date: Fri, 26 May 2023 12:37:48 -0400 Subject: [PATCH] Update PR template to include a11y checklist (#19538) --- .github/PULL_REQUEST_TEMPLATE.md | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index 110d1d819..17af1a953 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -49,10 +49,14 @@ _Please replace this line with instructions on how to test your changes, a note on the devices and browsers this has been tested on, as well as any relevant images for UI changes._ -### UI accessibility concerns? +### UI accessibility checklist +_If your PR includes UI changes, please utilize this checklist:_ +- [ ] Semantic HTML implemented? +- [ ] Keyboard operability supported? +- [ ] Checked with [axe DevTools](https://www.deque.com/axe/) and addressed `Critical` and `Serious` issues? +- [ ] Color contrast tested? -_If your PR includes UI changes, please replace this line with details on how -accessibility is impacted and tested. For more info, check out the +_For more info, check out the [Forem Accessibility Docs](https://developers.forem.com/frontend/accessibility)._ ## Added/updated tests?