@OneManBanned
Posted
Hi, the solution looks nice but I have a few suggestions.
-
your project links are all buttons and don't go anywhere. They should be <a></a> that point to the corresponding projects
-
Most of the images in your html would make more sense as background-images in css.
-
I would use <h2></h2> for the section headers - projects, contact - and turn the skills-container into a <dl></dl> .
-
The spans underneath your <h3>'s would be better as <li>'s inside a <ul>
I hope you find that useful.
well done, Brendan
Marked as helpful
@guiyee89
Posted
@OneManBanned Hey Brendan! Thanks very much for your feedback! Definitely very helpful since I am still trying to learn to make my HTML as clean as possible. I will make all this changes as soon as I can!
Thanks a lot! Guillermo