Skip to content

Refactor and add design

Justin Reese requested to merge justin-admin-review-refactor into main

main is protected so I can't commit directly to it.

This MR refactors the markup and JS of the admin review, and adds styling.

What it's missing:

  • Button styling. This is good enough to test the UX, we'll deal with buttons later.
  • Focus styling on the navigator buttons.
  • SVG check/x icons for navigator. We need to add those for accessibility -- red/green isn't color-blind-compatible -- but let's put this in LFC's hands first.
  • Requiring a comment with a "no" answer. Easy enough to add, let's see if it's desired.
  • Auto-checking the final checkbox when any question is failed.
  • Really, the whole final step needs to be understood from user testing and interview.
  • Any support for what to show when the review is complete.

Merge request reports