Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

hover effect and media screen

persanu13 100

@persanu13

Desktop design screenshot for the Social links profile coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Alex 3,130

@Alex-Archer-I

Posted

Hi!

Good job =)

It's a couple of semantic thing you can improve here.

At first - use a semantic tag in general. Like the main tag for the main content of the page (in this case it's the whole content), The h1 and p tags for texts.

And second - there is a list of links here. So you should use a tag for the links themselves and put them into the ul and li tags. Like this:

<ul>
    <li><a href="#">Link here!</a></li>
    <li><a href="#">Link here!</a></li>
    <li><a href="#">Link here!</a></li>
</ul>

Hope that helps. Neat work, keep doing =)

0

@Nrobke

Posted

hey @persanu13 it amazing. I like it

0

Please log in to post a comment

Log in with GitHub
Discord logo

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