Grid/Flexbox layout

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  Hi, Aleksandra Ryan! 👋

  Well done on this challenge! 👍

  A few things I suggest are,

  • Adding a max-width to the main container or wrapper of the page (such as div.container) and centering it in the viewport so that the layout of the page doesn't become too wide and the elements seem too spaced out on extra-large desktop screens.
  • Since you are adding more than one background image to the body in this challenge, I would suggest separating your background properties in your CSS (rather than try to fit many values into the single background property). This will make your styles easier for you and others to read and manage and should help with troubleshooting some issues (such as one of the background images repeating in the desktop layout of the page).
  • Rather than using absolute positioning to place the review boxes in the desktop layout, I think you can stick them to being placed by their flex container and simply using the transform property to align them differently instead. 😉

  Keep coding (and happy coding, too)! 😁

  Marked as helpful

  0