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
- For instance, Google the
- 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