My tribute page project to Mary Slessor - please comment/criticise

Code Feedback
1

http://codepen.io/tarboshi/full/ozopWY/

1 Like
2

Hi
This code is not correct:

<h1>Mary Slessor</h1>
<h4><strong>. . . . the mother of African twins</strong></h4>

Do not use lower levels to decrease heading font size: use the CSS font-size property instead.
Avoid skipping heading levels: always start from <h1>, next use <h2> and so on.

https://www.w3.org/TR/html5/sections.html#the-h1,-h2,-h3,-h4,-h5,-and-h6-elements

h1–h6 elements must not be used to markup subheadings, subtitles, alternative titles and taglines unless intended to be the heading for a new section or subsection. Instead use the markup patterns in the Common idioms without dedicated elements section of the specification.

Common idioms:
https://www.w3.org/TR/html5/common-idioms.html#common-idioms

HTML inspector:

  • The <p> element cannot be a child of the <ul> element.
  • The ‘alt’ attribute is required for <img> elements.

Cheers and happy coding :slight_smile:

3

All I noticed was that her surname is starting with a lower letter in a couple of places (her quotes) other than that, to me looks very pretty and definitely done for a woman worth a tribute page :thumbsup: