Musician Survey - Survey FCC Form

Hello,
So I did this form when I was doing the html certificate. Now I have it, but I would like some criticism to see what I should fix in my previous projects.

So any criticism or suggestions would be welcome.

Thanks

Hi,
When I fill out the age field, it tells me “please select a value that is no more than zero.”

I tried a negative number, but it can’t be less than zero. So age can only be zero. :slight_smile:

Other than that, it seems to work ok, and looks fine.

1 Like

Your form looks good @anon43350554. 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 HTML coding errors you should address.
  • Codepen provides validators for HTML, CSS and JS. Click on the down arrow in the upper right of each section and then click on the respective ‘Analyze’ link.
    • The one for HTML misses things which is why I recommend W3C
    • The one for CSS is good. Use it, there’s something to clean up.
  • Don’t use <br> to force line breaks or spacing. That’s what CSS if for.
  • Change the cursor to a pointer when hovering over the submit button.
  • Is this (font-family: cursive monospace serif;) really a font name? Or should there be some comma’s in there?
1 Like

Thanks, I fixed this by removing the max value for the number input.

Thanks.
I fixed all these issues.

It took time to be honest.
I also had to rewrite all the css because there was a bug in it.

Thanks for your help. And I will be using w3c validator for all my projects.

Nice job cleaning everything up. There’s still one <br> that you can delete.

1 Like