Skip to content
Submitted 3 months ago

Social links profile using flexbox and media query

voespinal-code•50
@voespinal-code
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 are you most proud of, and what would you do differently next time?

This time it was a little easier for me to accomplish this challenge.apply other techniques

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

simplify my code if possible

Community feedback

  • Lance•1,580
    @IzykGitPosted 3 months ago

    Good work!

    The best way that you could simplify this code would be to use less divs and more semantic HTML elements. For instance you have your list with the links that are wrapped in a div element and then contains p elements.

    A better way to do this would of been to use either a <ul> element (unordered list). This element would allow better context for the browser, improve SEO, as well as better defining the structure.

    For the main content it is good that you used a main element however this is really all you needed. Added a child div for the container and then another child div for the card is just adding bloat to your code.

    Keep up the good work! :)

    Marked as helpful
    0
    voespinal-code•50
    @voespinal-codePosted 3 months ago

    thank you bro.I will listen to your advice and I will try to do what you tell me. @IzykGit

    0
  • Vincin Christmas•150
    @VincinChristmasPosted 3 months ago

    good job

    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