Review my Responsive Web Design projects

Project Feedback
#1

Link to my pens

I hope it’s okay that I am posting a bundle. You don’t have to review all projects, last few or only the last one would be fine.

I would appreciate any advice. If there is anything that I could be doing better, please tell me.

#2

Welcome to the forums @mayolk. Your projects look good. Some things to revisit;

tribute page

  • 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.
    • “wikipedia page” is not accessible
  • it looks as though the font-family fonts were copied from the sample code. First off, don’t do that. Second, there are fonts that are not called properly because there is no link to them.

A couple of suggestions;

  • on smaller screens the .text paragraphs stretch to the limit of the container. adding some padding/margin would look better
  • on smaller screens the bullet points are outside the container

survey form

  • Run your HTML code through the W3C validator.
    • There is an HTML syntax/coding error you should be aware of and address.
    • Since copy/paste from codepen you can ignore the first warning and first two errors.
  • A lot of the code looks to be the same as the sample project with only the display text changed.
    • Make the project from scratch, with your own code, style and content. Don’t take code from the sample project.

product landing

  • 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.
  • 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)

tech doc

  • Run your HTML code through the W3C validator.
    • There is an HTML syntax/coding error you should be aware of and address.
    • Since copy/paste from codepen you can ignore the first warning and first two errors.
  • It’s a nit but you should cite your reference

portfolio

  • 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>’
#3

Hi,

I am sorry if it looks as if I have copied code from the sample projects, but I didn’t. I wasn’t very inspired for the Tribute page, and Survey page was an important practice for me so I didn’t bother much to make it look unique. But I assure you, I wrote everything from scratch, I only copied some singular CSS rules, never an entire ruleset, and I haven’t copied any HTML. Also about the Technical Documentation page, “Super Strong CSS” does not exist, I made that up, so there is nothing to reference there.

I will implement all your advice, and I will run all the code trough the W3C validator, I had no idea I was making mistakes in every HTML.

Thank you.

#4

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