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

i used html for structure and flex box and buttons for styling

Danish 80

@Danish030

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

@temesgen-982

Posted

  • Semantic HTML You probably should be using link elements <a> instead of buttons for the social media links since you want them to take the user into another page.
  • Font Size The other one is you should use the font size specified on the style guide for the paragraph elements.

Marked as helpful

1

@Saul-Gustavo

Posted

Hello, I saw your code and there are some things you can take into account to improve your project, in the body, you can use min-height 100vh and center the container with flexbox.

Inside the container you can use the padding property to give more padding to the content, this way you would avoid using the height and width property. You can do it, keep it up.

I would also recommend using semantic html, it will make your code look clean and easy to understand.

Marked as helpful

0

Danish 80

@Danish030

Posted

thank you, i will improve and work on your recommendations @Saul-Gustavo

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