1 Like
Your form looks good @emilymeredith. Some things to revisist;
- When using codepen it only expects the code you’d put within the
<body> </body>
tags in HTML. (No need to include the body tags). For anything you want to add to<head>
click on the ‘Settings’ button, then HTML and add it into the ‘Stuff for <head>’ box.- Mentioned because you have
div
element outside ofbody
. That is not correct. Just get rid of thebody
tags in codepen.
- Mentioned because you have
- 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 clean up.
- Don’t use
<br>
to force spacing or line breaks. Use margin and/or padding in CSS - Users should be able to click on the labels to select, not just the radio buttons and checkboxes.
- Review the lessons on creating radio buttons and creating checkboxes.
- ‘Comments’ should be a placeholder in your textarea. Don’t make users have to delete text that you’ve added.
- Change the cursor to a pointer when hovering over the submit button
- Make your form responsive. You’ve got all sizes hard-coded in pixels. You don’t want that or you won’t be able to make it responsive.
- On a side note, keep the test script when forking the pen (
<script src="https://cdn.freecodecamp.org/testable-projects-fcc/v1/bundle.js"></script>
).- The test script should be included, with all tests passing, when you submit your projects.
2 Likes
Thanks for the feedback! I’ll look into those issues.