Skip to content
Submitted 5 months ago

Social links profile

craftzniac•120
@craftzniac
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Social links profile
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

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

  • craftzniac•120
    @craftzniacPosted 5 months ago

    thanks for the feedback

    0
  • geomydas•1,440
    @geomydasPosted 5 months ago

    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