I just finished project 5 - personal portfolio. I completed all the user stories and I made it responsive (mobile friendly).
I’m sure there is a lot that could be improved. I took a minimalist approach. Some of my margins are a little crazy (last page)…but it worked lol
I would love to hear your feedback…be as brutally honest as you can - thank you!
Looks good however its creating a scroll bar at the bottom on mobile view.
Your portfolio looks good @Stampflmail92. Some things to revisit;
- Run your HTML code through the W3C validator.
- There are HTML 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)
- Keep all styling external. Do not use in-line styling.
- Make your page responsive. Remember, the R in RWD stands for Responsive
- There’s a horizontal scrollbar on smaller screens
Check paddings and margin. The nav has 100% width, but the main content sector - not. Looks not good.
Check buttons. When you hover it - they are moving. And it looks like not a feature