@jamesbarnett
Posted
I think you did a great job in regards to semantic HTML. It did not occur to me use h1
and h2
for the Name and Location respectively, but it seems obvious looking at your code. Clean, simple, nicely done!
I ran the WAVE accessibility plugin on your site, and there were no issues. Great work in this regard, too.
Checked on a range of screens, it is very nicely responsive and handles small screen sizes quite well.
I found the code very readable and well-structured personally. I did run the code through the html-validator tool, and it did complain about the trailing slash in your single element coding tags. The tool claimed these can create problems if used with unquoted attributes. Sounds like quite the edge case to me.
Marked as helpful
@FengDenny
Posted
@jamesbarnett
Really appreciate your comment! Thanks for letting me know the trailing slashes
edge case. Happy front-ending, Jim!