FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

add a11y checklist to pr template · devhttps/frontend@a3dbb25 · GitHub

Commit a3dbb25

Browse files
committed
add a11y checklist to pr template
1 parent 433a45f commit a3dbb25

1 file changed

Lines changed: 21 additions & 7 deletions

File tree

‎.github/PULL_REQUEST_TEMPLATE.md‎

Lines changed: 21 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,17 +2,31 @@
22

33
## What is the value of this and can you measure success?
44

5-
## Does this affect other platforms - Amp, Apps, etc?
5+
## Checklist
6+
7+
### Does this affect other platforms?
8+
9+
- [ ] AMP
10+
- [ ] Apps
11+
- [ ] Other (please specify)
12+
13+
### Does this affect GLabs Paid Content Pages? Should it have support for Paid Content?
614

7-
## Does this affect GLabs Paid Content Pages? Should it have support for Paid Content?
815
<!-- if there are versions of this content with the paid styling (teal and grey) then they will need to be checked -->
916
<!-- content can be found here: https://www.theguardian.com/tone/advertisement-features -->
1017

18+
- [ ] No
19+
- [ ] Yes (please give details)
20+
21+
### Accessibility test checklist
22+
23+
<!-- for changes that affect how a page appears in the browser -->
1124

12-
## Screenshots
25+
- [ ] [Tested with screen reader](https://accessibility.gutools.co.uk/testing/web/screen-readers/)
26+
- [ ] [Navigable with keyboard](https://accessibility.gutools.co.uk/testing/web/keyboard-navigation/)
27+
- [ ] [Colour contrast passed](https://accessibility.gutools.co.uk/testing/web/colour-contrast/)
1328

14-
## Tested in CODE?
29+
### Tested
1530

16-
<!-- AB test? https://git.io/v1V0x -->
17-
<!-- AMP question? https://git.io/v9zIE -->
18-
<!-- Does this PR meet the contributing guidelines? https://git.io/v1VEJ -->
31+
- [ ] Locally
32+
- [ ] On CODE

0 commit comments

Comments
 (0)

Back | FazBrowse Home | New Git URL