Code Review Checklist
- Are elements on the page aligned appropriately?
- Are functions and other names within the code descriptive and accurate?
- Is it responsive? (desktop, tablet, mobile)
- Did you check with a viewport more than 1400px wide?
- Is there CSS using existing or new helper classes where appropriate?
- Are the CSS rules specific enough such that they don’t affect other elements across the site?
- Are CSS rules only as specific as they need to be?
- Did you check Safari?
- Are there sizes on all SVGs?
- Are images wrapped in DIVs?
- Are there no console errors?
- Has the theme/plugin version been updated? (Hint:
yarn run version patch) - Are images appropriately sized?
- Is content (images, text, buttons, etc.) translated (if applicable)?
- Are CSS breakpoints consistent?
- Is critical functionality above 95% support or can be polyfilled by postcss?
- Have you checked on an actual device?
- Are styles in the stylesheet whenever possible? (ie. not in the style attribute)
- Does it work with touch?
- Is other cross-dependent code merged or ready to be merged?
Future Items
- Did you do a visual diff?
- Are appropriate JS and PHP library functions being used?
# Code Review Checklist
- [ ] Are elements on the page aligned appropriately?
- [ ] Are functions and other names within the code descriptive and accurate?
- [ ] Is it responsive? (desktop, tablet, mobile)
- [ ] Did you check with a viewport more than 1400px wide?
- [ ] Is there CSS using existing or new helper classes where appropriate?
- [ ] Are the CSS rules specific enough such that they don't affect other elements across the site?
- [ ] Are CSS rules only as specific as they need to be?
- [ ] Did you check Safari?
- [ ] Are there sizes on all SVGs?
- [ ] Are images wrapped in DIVs?
- [ ] Are there no console errors?
- [ ] Has the theme/plugin version been updated? (Hint: ```yarn run version patch```)
- [ ] Are images appropriately sized?
- [ ] Is content (images, text, buttons, etc.) translated (if applicable)?
- [ ] Are CSS breakpoints consistent?
- [ ] Is critical functionality above 95% support or can be polyfilled by postcss?
- [ ] Have you checked on an actual device?
- [ ] Are styles in the stylesheet whenever possible? (ie. not in the style attribute)
- [ ] Does it work with touch?
- [ ] Is other cross-dependent code merged or ready to be merged?