@Theosaurus-Rex
Posted
Hey @Madeline0421, great work! Looks very close to the design!
A couple of suggestions for your HTML that will make it more accessible:
- The image on this card is purely decorative, so you can actually leave the alt text empty to avoid additional noise for screen reader users! Using
alt=""
is perfectly acceptable here. - In your markup, you're using a lot of divs that could be replaced with more meaningful elements
- The tag holding the date could be changed to a
<time>
tag - this tag can be provided adatetime
attribute, which takes in the publishing date in a machine-readable format. In a real-world scenario, this will help search engines to rank your website content! You can read more about this tag on W3Schools if you'd like to learn more. - For text content such as the body text of this card, a
<p>
tag is preferable, and denotes hierarchy within the markup content - The alt text on your author image isn't very descriptive - if you'd like to provide alt text here, try describing the picture like you would to someone who cannot see it, e.g. "a headshot of a man standing against a plain white wall"
- The tag holding the date could be changed to a
Otherwise, awesome work! I hope this helps!
Marked as helpful
@Madeline0421
Posted
@Theosaurus-Rex Thank you for your advice! Will definitely incorporate these more accessible/hierarchal practices. :)