If you could evaluate my survey form, please.
Thank you!
Hey, just a few things to point out:
- The label on the select drop down isn’t working correctly. The
for
attribute on the label needs to point to theid
on the select. Same issue for the textarea at the bottom. - None of the labels on the radio buttons/check boxes are working correctly. You’ll need to give those inputs
id
s so you can set thefor
attribute properly on the labels. - Don’t use the
br
element to create new lines between elements. Thebr
element does have some legitimate uses (not many though) but using them to create new lines between elements is not one of them. Use CSS to do this instead. You might also need to add helper divs, which is perfectly acceptable. - The primary content of your page, which is everything between the
header
andfooter
should be wrapped in amain
element. This is a best practice for accessibility. - I’m seeing a consistent horizontal scroll bar at all widths. Granted, it doesn’t scroll much, but I think you should get rid of it.
But overall, great job. We don’t get to see a lot of finished projects like this since FCC moved to the new format. Thanks for sharing.
This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.