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

Includes Sass implementation and Grid ,Flex layouting

@Kevin-kirago


Design comparison


SolutionDesign

Community feedback

@Kevin-kirago

Posted

Just updated my solution. Thanks for the feedback!

0
P
Matt Studdert 13,611

@mattstuddert

Posted

Nice work on this challenge, Kevin! Just a quick heads up that the min-height: 100vh; on the .header__container class is throwing off the screenshot. You can edit that and take a new screenshot to resolve it. vh is a unit that should be used sparingly as it can create unintended visual issues. Instead, I'd recommend using padding and margin on elements within to create the necessary spacing.

Overall your code looks really good. Keep it up! 👍

0

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