Final Project Feedback HTML-CSS

Code Feedback
1

Hello everyone. I’ve have finished my last project about “Personal Porfolio”. I really hope to hear feedbacks from you guys. Thank you!

Link: https://codepen.io/TechnologyFuture/pen/xxdaeLb

Oh and also, please check the contact place especially the Youtube one because I think there’s a problem there.

2

Hi!
Its great .I still didn’t reached this project. Thanks for giving me some ideas.

3

No problem. Looking forward to see your project ^^

4

Hi @phamminhphu161009
Your hover ideas are really nice.
specially on cards.

5

Thank you. The idea was from the FCC team by the way. I used it on my menu bar as well and I think it’s fine. Anyway, thank you ^^

6

Your page looks good @phamminhphu161009. Some things to revisit;

  • 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.
    • 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)
  • Do not use the <br> element to force line breaks or spacing. That’s what CSS is for.
  • The link to the survey form gives uses a 404 error
  • overflow-x: hidden; will completely hide anything that extends outside its container element horizontally. It’s recommended to not do this, and refactor your CSS to be more responsive - otherwise it is possible for your site to appear to be missing content.
8

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