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

Insure Landing Page

Mar 220

@Mar5Planet

Desktop design screenshot for the Insure landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


My question is this- I'm a fairly new developer and want tips on how I write my CSS code- am I writing too much CSS to make it responsive? maybe things will get easier once I learn Javascript better

Community feedback

P
Matt Studdert 13,611

@mattstuddert

Posted

Hey Mar, overall you've done a really good job! There are parts in your CSS where you could remove some code duplication. But learning to refactor your code and clean it up will come with time. Just look for any common styles that you're repeating and move them to a single class that can be reused.

Another thing that should reduce the amount of CSS code you write is to switch to using min-width media queries instead of max-width. Using min-width media queries often leads to less CSS code. It also has the benefit of loading in fewer styles for mobile users which can be a performance gain.

I hope this helps. Keep up the great work! 👍

1

Mar 220

@Mar5Planet

Posted

@mattstuddert thank you!!

0
micky 400

@awwmicky

Posted

Dang, the responsiveness is good 📦! The only feedback I can give is to figure out how to align the footer text & footer social icons centers. That is a small fix I see in desktop mode, when I shrink the page.

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