Always convert tabs to spaces
Use consistent tabbing structure
HTML5 semantics suggest a different way of dealing with header tags in your document -- http://webdesign.tutsplus.com/articles/the-truth-about-multiple-h1-tags-in-the-html5-era--webdesign-16824
Only <li></li> can be nested within a <ul>
<i> is used for editing the style of a particular set of words in a paragraph. This is a paragraph block -- use css to style this as italic. (Either <p> or <div> are fine)
pay attention to consistency
need space between tag attributes
Close tags that require closing
When placing tags on multiple lines, be consistent. All lines within the block should share that format.
in HTML, these extra spaces will cause issues in formatting and layout. Do not have extra spaces.
This is redundant. Use a class of 'button' when styling an <a> tag as a button.
Buttons should be used for forms, and possibly triggering javascript functionality. <a> tags should be used for links.
'./' tells the filesystem to 'start looking from my current location' always use a proper prefix, when using relative paths.
This semantics seems really odd. I would suggest something closer to this.
- Comments are articles ( http://www.w3.org/TR/html5/sections.html#the-article-element )
- <a> tags should not wrap <h#> tags.. Proper structure: <h#> tags wrap <a> tags
- I should be able to click the avatar image as well (semantic ux)