Review my Survey Project

Code Feedback
#1
#2

This is really nice. I like to test for accessibility and you did a great job. There are a few minor points I will list below, but overall this is excellent.

  • Technically, the form should be wrapped in a <main> since it is the main content of the page.
  • You are relying on the browser’s default focus outline, which does technically pass current WCAG standards (for now) but I always recommend that you style the CSS outline property for the :focus pseudo-class on all inputs so that it looks consistent across all browsers.
  • Really like that you used rem units for almost everything. Now extend that to the max-width on the form so it will widen if the user manually increases the font size.

Great job!

1 Like
#3

Actually working on this project now, Great job.

1 Like
#4

Thank u for the feedback…

#5

This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.