Skip to content
Submitted about 3 years ago

Resposive Layout Using CSS Grid

Matt Cowan•40
@mattbcowan
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Huddle landing page with a single introductory section
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Any feedback would be great! Everyone has been amazing so far. Thank you all for your feedback on these challenges!

Community feedback

  • Ken•4,915
    @kens-visualsPosted about 3 years ago

    Hey @mattbcowan 👋🏻

    I have some suggestions for the project.

    • First, in desktop viewport width, remove margin-top for footer, because it adds some unnecessary scrolling to the page.
    • Next, even though it doesn't look like a list, footer icons should be in ul and each icon could be in li which in it has a tag.

    I hope this was helpful 👨🏻‍💻 you did a superb job with this one, no accessibility or HTML issues, and it's responsive, keep that up. Cheers 👾

    1

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