hey everyone, just finished the survey page,would love to get some feedback, thanx
Welcome to the forums @prarthana1504. Your form looks good on desktop. Some things to revisit;
- Codepen provides the boilerplate for you. It only expects the code you’d put within the
body
element in HTML. (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.- The link to the font goes in the box labeled ‘Stuff for <head>’
- Mentioning because you have elements out of order. Everything the browser renders must be in the
body
element. You have adiv
element outside of it.
- Run your HTML code through the W3C validator.
- There are HTML coding errors you should be aware of and 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 CSS is good. Use it, there’s a typo to clean up.
- The one for HTML misses things which is why I recommend W3C
- User’s should be able to select by clicking the label, not just the radio button / checkbox.
- Don’t use
<br>
to force line breaks or spacing. That’s what CSS is for. - Make your page responsive. It falls apart on smaller screens.
- as a hint, you don’t want to use absolute pixel values for padding / margin. Use a percentage instead.
1 Like
@Roma thank u for the insight, im really new to making wepages and hadnt considered these points, ill look into correcting the code.