Submitted over 3 years ago

Responsive - CSS - SASS

Andrea• 270

@AndreaLecuona

Preview site View code

This is a solution for...

Insure landing page

HTML
CSS
JS

2junior

View challenge

Design comparison

SolutionDesign

Community feedback

banana joe• 135

@bananaJoeReal

Posted over 3 years ago

Hey Andrea Lecuona!

Your solution looks nice but it's not responsive.

Change the break point 375px to 620px it will be great! (For the header).
Add hover to the header links to be interactive with the user.

Happy codding!

Andrea• 270

@AndreaLecuona

Posted over 3 years ago

@bananaJoeReal Thanks for you feedback! I'll be fixing it very soon. Have a great day!

Jadson Santos• 235

@santos-jadson

Posted over 3 years ago

Hello Andrea, how you doing?

Your code is really good and very similar to the original design. I don't have anything bad to say, I mean, the only thing I think you could change is the responsiveness, because when I open the site in a resolution bigger than 375px, the layout is breaking. But don't worry, your code is great and that's not a huge issue :D.

Andrea• 270

@AndreaLecuona

Posted over 3 years ago

@santos-jadson Thanks for you feedback! I was so worried about getting the 1440 and 375 resolutions right, that I overlooked the intermediate sizes. Have a great day!

Please log in to post a comment

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