This project will test your layout skills.

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted about 2 years ago

  Hello Mohamed, congratulations for your solution!

  Your solution seems great, but there's a room for improvement, for example, you can start by improving the html markup replacing the div for rating section and cards with <article> and the quote inside the card with <blockquote>.

  The container size correct is max-width: 1110px and you don't need many media queries just make the container responvsive replacing width with max-width to allow the container and cards fit the screen resizing automatically.

  @media (min-width: 1200px) { 
       .container { 
           width: 1175px; 
       }
  

  Hope this helps, happy coding!

  Marked as helpful

  1

  ---

  

  Mohamed Khaled•190

  @mohamedKhaled89Posted about 2 years ago

  @correlucas thanks alot for your feedback

  • I really need to improve my using of media queries with responsive layout.

  0