Feedback Needed......survey project

Code Feedback
1

Hello everyone, here is my survey form project. need your precious feedback

2

Wow, this looks really nice! The only thing that I would do would be to add a small border radius around the form element with the id of survey-form (on the 44th line of your css code). (also where did you get that background image??? It looks really nice!)

3

Thank you for your feedback and yeah really nice image. I just googled it.

4

Your form looks good @sayeed_mostafa. 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.
    • Your page passes 14/17 user stories. Click the red button to see which test(s) are failing and text to help you correct the issue.
    • Be sure and read more than just the first line of the failing message. The ability to read and comprehend error messages is a skill you’ll need to acquire as a developer. Ask questions on what you don’t understand.
  • Codepen provides the boilerplate for you. It only expects the code you’d put within the body element in the HTML editor. (No need to include the body tags). For anything you want to add to the <head> element click on the ‘Settings’ button, then HTML and add it into the ‘Stuff for <head>’ box.
    • For instance, links to fonts go in the box labeled ‘Stuff for <head>’
  • Run your HTML code through the W3C validator.
    • There are HTML syntax/coding errors you should be aware of and address.
  • Codepen provides validators for HTML, CSS and JS. Click on the chevron in the upper right of each section and then click on the respective ‘Analyze’ link.
    • The one for CSS is good. Use it and address the issue(s).
    • (The one for HTML misses things which is why I recommend W3C)
  • Do not use the <br> element to force line breaks or spacing. That’s what CSS is for.
  • User’s should not have to delete text when entering comments. This is a bad UX.
  • Change the cursor to a pointer when hovering over the submit button
  • Placeholder text should not mirror the label. It should show the user the format they are expected to input.

On a side note, do not put links in the topic title. They cannot be clicked and do not convey any helpful information. (I’ve edited the title to remove the link)

5

Thank you for your precious feedback.

6

This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.