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

Responsive Grids and Bootstrap

@Hany-soliman


Design comparison


SolutionDesign

Solution retrospective


Hey Guys!

This is my first challenge after finishing the CSS section of "The Complete 2020 Web Development Bootcamp". I feel that I've improved since the first time I submitted a solution here. But I'm not satisfied yet, so I'm looking forward to your feedback.

Thanks! Hany

Community feedback

P
Matt Studdert 13,611

@mattstuddert

Posted

Hey Hany, that's great to hear that you feel like you've improved since going through the CSS section of the course you're taking. Your solution looks really good! I've got a couple of small suggestions related to your CSS:

  • I'd recommend avoiding using IDs for styling purposes. They're too specific and can't be reused on the page. Instead, I stick to attribute, class, pseudo, and type selectors to make your code more maintainable.
  • Using min-width instead of max-width media queries can lead to less code. They also reduce the CSS that mobile users load-in, which is a performance benefit.

Keep up the great work! 👍

0

@Hany-soliman

Posted

@mattstuddert thank you! I will work on it and update the solution.

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