Please view, comment, criticize on my pen and help make suggestions
link to pen:
Welcome to the forums @ebujakes.
When asking for feedback on your project you’ll get more eyes-on in the #project-feedback subforum. I’ve taken the liberty of moving this for you.
Your pages look good. Some things to revisit;
tribute
- 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.
- There are HTML syntax/coding errors you should be aware of and address.
- Do not use the
<br>
element to force line breaks or spacing. That’s what CSS is for.- Reference MDN Docs
- Accessibility is about being accessible to all users. Review the giving meaningful text to links lesson. For a more thorough explanation read Web Accessibility in Mind.
- “clicking” is not accessible
- Make your page responsive. Remember, the R in RWD stands for Responsive
- There’s a horizontal scrollbar on smaller screens and text is cut-off
survey
- 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 thebody
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. - 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)
- The one for CSS is good. Use it and address the issue(s).
- Do not use the
<br>
element to force line breaks or spacing. That’s what CSS is for.- Reference MDN Docs
- User’s should be able to click on the label to toggle a selection, not just the radio button / checkbox
- Make your page responsive. Remember, the R in RWD stands for Responsive
- There’s a horizontal scrollbar on smaller screens
Thanks for the review. Much appreciated
This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.