Hello everyone, I’ve successfully completed survey-form project and I’d appreciate feedback for it. Thank you in advance.
Here’s the link:
Survey-form project
hey @theaaroon. nice survey form! i’m not experienced enough to review the code, but here are some things about the page itself i noticed.
- on the age option it allowed me to type the letter e. i’m assuming you wanted numbers only, so i figured i’d let you know
- it’s a bit tough to determine what the background photo is (at least for me it was). maybe consider changing it or having a solid background instead?
all in all, though, the form is good! nice work
EDIT: if you want a way to easily check your code, here’s a code validator https://validator.w3.org/
Welcome to the forums @theaaroon. Your form looks good. Some things to revisit;
- Keep the test script when forking the pen (
<script src="https://cdn.freecodecamp.org/testable-projects-fcc/v1/bundle.js"></script>)- The test script, with all tests passing, should be included when you submit your projects.
- Run your HTML code through the W3C validator.
- There are HTML syntax/coding errors you should be aware of and address.
- Since copy/paste from codepen you can ignore the first warning and first two errors.
- Your form says age is optional but it’s a required field. This is a bad UX
- Change the cursor to a pointer when hovering over the submit button
- It’s a nit but the placeholder text should not mirror the label. Rather it should give the user the format of what’s expected.
On a side note, do not replicate the sample projects. The instructions say yours should be functionally similar and to give it your own personal style.
@ecouch23, I don’t want to hijack the OP’s topic. Research and find out why this is not an issue.
1 Like
This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.