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

Responsive social link profile using angular and flex

P
Amit-R328 150

@Amit-R328

Desktop design screenshot for the Social links profile coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


What specific areas of your project would you like help with?

Any feedback is appreciated

Community feedback

Darkstar 1,000

@DarkstarXDD

Posted

Looks nice. I have couple of feedback.

  • The <header> or the banner role is usually used to wrap content that is common across multiple pages of a website. Usually you see it at the top of the webpage wrapping the site logo and a navbar. I would not use a <header> in this challenge.
  • What you need is a <main> landmark element. Every web page needs a <main> element wrapping the main content of the page.
  • The <nav> element is for a list of links that navigate within the site. Not for links that navigate to outside websites such as social media sites. In this case having the links inside a <ul> is enough.

Marked as helpful

1

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