Skip to content
Submitted 7 months ago

Responsive landing page using flex, html and css

Julie Crotzer•360
@JCrotzer
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?

Just working on fun projects to enhance my coding skills.

What challenges did you encounter, and how did you overcome them?

N/A

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

None

Community feedback

  • Moshiur•230
    @codermoshiurPosted 7 months ago

    Excellent work. For links, I believe the anchor tag would be more appropriate than the p tag.

    1
  • Tharun Raj•1,330
    @Code-BeakerPosted 7 months ago

    Hi, congratulations on finishing this project! It is a great work.

    Here are some of my recommendations regarding your solution:

    Instead of using <div> for the link and p for the text inside the link, like this:

    <div class="link">
<p>GitHub</p>
</div>

    You can use an anchor tag so it is more accessible and so that the element matches its original purpose. You can use this method:

    <a class="link">GitHub</a>

    This will help you minify your CSS by styling an anchor tag instead of a <p> inside a <div>.

    Hope this helps you. Happy Coding 🎉

    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