Skip to content
Submitted about 1 month ago

flex

Babatunde Owolabi•80
@nottundeednut
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

Community feedback

  • Kamran Kiani•1,280
    @kaamiikPosted about 1 month ago

    Hi. I have some notes I wanna mention:

    1. Your links take you to a new page. So you better use a tag instead of button. Also the links are like a list of items. So you can use ul with li for them.

    2. Use a strong CSS reset for your CSS. Andy Bell's is good.

    3. For the font-size, You have to use rem for unit instead of px. Here explains why . Also for padding and margins you can use em or rem.

    Marked as helpful
    0
  • rishavdhungel•70
    @rishavdhungelPosted about 1 month ago

    Looks great, it would be better to have a <main></main> container in the body and put everything within it.

    <body>
    <header>
        <!-- Header content -->
    </header>
    <main>
        <!-- Main content goes here -->
    </main>
    <footer>
        <!-- Footer content -->
    </footer>
</body>
    Marked as helpful
    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