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
outlineproperty for the
:focuspseudo-class on all inputs so that it looks consistent across all browsers.
- Really like that you used
remunits for almost everything. Now extend that to the
max-widthon the form so it will widen if the user manually increases the font size.
Actually working on this project now, Great job.
Thank u for the feedback…
This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.