@mattstuddert
Posted
Hey Artur,
Great job on this challenge! Your project is definitely a good representation of the design! Here are a couple of pointers I thought about after looking at your HTML code:
- You don't have a
h1
on this project as it stands. I'd always recommend having a singleh1
. For this project, it would be where you've got theh2
headings. Except, I'd have it all as a singleh1
withspan
elements inside to differentiate the styles. This means the cards headings would becomeh2
s. - For the images, I'd keep the
alt
value empty. This will make screen readers skip over them. In this instance, that's a good thing, as thealt
text is currently repeating the exact text in the card headings and adds no extra context.
I hope these tips help. Keep up the great work! 🙂