Survey form fcc feedback please

Hi, just completed my second project. I would appreciate if you take some time to give me your valuable feedback.
Survey Form FCC

Hi @daiwik,

I think your form looks great! The black & white background image goes well with the color of the form.

I noticed that your submit button doesn’t change the cursor to a pointer when hovering. I think adding this would help convey that the submit button is clickable. You can change that in your CSS by adding

cursor: pointer;

So your submit button CSS would look like:

input[type=submit]{
  cursor:pointer;
  width:100%;
  border:none;
  padding:10px;
  font-weight:bold;
}

Just a small suggestion, as otherwise everything looks great!

Good luck & happy coding!

1 Like

Your form looks good @daiwik. Some things to revisit;

  • Run your HTML code through the W3C validator.
    • Since copy/paste from codepen you can ignore the first warning and first two errors.
    • There are coding errors you need to address.
  • Don’t use <br> to force line breaks or spacing. There are ways to group your like items.
    • For instance, Google the fieldset element
    • Use margin and/or padding in CSS for adding spacing
  • This is line 68 in your code, <input type="submit" vaue="submit" id="submit">
    • Spot the typo for one of your attributes
  • Change the cursor to a pointer when hovering over the submit button
1 Like