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

social-links-profile-main

@stadnik-petro

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

jkaps9 140

@jkaps9

Posted

HTML is missing the main tag which is standard in HTML5.

Your screenshot doesn't seem to match your final solution

Marked as helpful

1

@stadnik-petro

Posted

@jkaps9 Thank you for your thorough comment!

0
P
AKdeBerg 160

@AKdeBerg

Posted

The code looks great. In my code, I used div for the links. Here you used nav tag which is more appropriate, I think.

Marked as helpful

1

@stadnik-petro

Posted

@AKdeBerg Thank you for commenting on my work, I think you were right to use div instead of nav, because this is a card and not a website, in the navigation bar of a website you should use nav, but now I look and think that in the card my decision was wrong.

1
P
AKdeBerg 160

@AKdeBerg

Posted

@stadnik-petro I see! that's a really helpful tips. thanks

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