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

Resposive Layout Using CSS Grid

@mattbcowan


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

P
Ken 4,915

@kens-visuals

Posted

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
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