News homepage using CSS grid

Community feedback

• 

  P

  Josh Kahlbaugh•380

  @Joshk7Posted 12 days ago

  Great job! The proportions of the solution match the design very well. However, setting the explicit width on the main element is causing the main content to overflow outside of the viewport width of the body element.

  .container{
      font-size: 1.5rem;
      width: 144rem;
      margin-left: 19rem;
      margin-right: 19rem;
      margin-top: 5rem;
      
  }
  

  Instead, you could set width to 100% and that would be a good start since explicit widths often make building responsive websites more difficult.

  Hope this helps!

  0

  ---

  

  saccoViola•90

  @saccoViolaPosted 11 days ago

  @Joshk7 hi and thank you for your feedback! I tried to set the width of the container at 100% but it get too large! I want to keep the width of 1440px, where should I set this value then? Thank you so much :)

  0