Mike• 80
@MikeyOnyedika
Posted
thanks for the feedback
0
Design comparison
Solution retrospective
What specific areas of your project would you like help with?
Can you help me point out where I'm not using semantic HTML properly? Thanks
Community feedback
geomydas• 1,060
@geomydas
Posted
I don't know shi about JS but this part " <section class="card">" shoud be a <main> tag not a section tag. You would only do this if you have low text content on your website (such as the newbie projects on frontendmentor)
<p>Jessica Randall</p> <span> London, United Kingdom </span>The p tag would be better with h1 or h2 since the screen readers will treat it like any paragraph without focus. The span doesn't give any meaning at all and you should replace it with an h2 or h3 tag
0
Please log in to post a comment
Log in with GitHub
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord