@dayashton This is very nice! I like the fade animations. Your code is well-structured, concise, and easy to read. It’s clear that you have a strong understanding of HTML, CSS, bootstrap, and jQuery. Here are a few ways I think this can be improved:
Your image has an alt attribute, but it’s redundant. It’s ok to drop the
Picture of part because the img tag tells screen readers that it’s a picture. It may even be possible for you to leave it empty
alt="", although I’m not totally sure about that. You’ll want to read WebAIM’s guidance on this to be sure.
The alt attribute should typically:
Be accurate and equivalent in presenting the same content and function of the image.
Be succinct. This means the correct content (if there is content) and function (if there is a function) of the image should be presented as succinctly as is appropriate. Typically no more than a few words are necessary, though rarely a short sentence or two may be appropriate.
NOT be redundant or provide the same information as text within the context of the image.
NOT use the phrases “image of …” or “graphic of …” to describe the image. It usually apparent to the user that it is an image. And if the image is conveying content, it is typically not necessary that the user know that it is an image that is conveying the content, as opposed to text. If the fact that an image is a photograph or illustration, etc. is important content, it may be useful to include this in alternative text.
<p class="text-center text-muted lead">, you can use the
blockquote element. I think that’s more descriptive: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/blockquote
The only other thing I notice is that you may be able to use some of the HTML5 semantic elements like
That’s all I see. The rest looks perfect to me.