single price grid component with flexbox

Community feedback

• 

  Mijail Hernandez•950

  @MishaHernandezPosted almost 4 years ago

  Hi Tyler!

  1. Container margins are left over if you are already using flexbox for absolute centering (justify-content: center and align-items: center). But that absolute centering you're doing doesn't work properly because you need to assign its container (body) a height: 100vh.

  2. The blocks at the bottom (.sign-up and .benefits) are in columns but they don't cover the entire row because they still retain a width of the 50% that you assigned them, so you must rewrite that value to 100% within the media query.

  3. I encourage you to review the documentation about Flexbox and Grid, the latter provides a more comfortable way to solve this design.

  Greetings and continue with the challenges :)

  2

  ---

  

  Tyler Case•90

  @tcase629Posted almost 4 years ago

  Thank You.

  0
• 

  Guliye•290

  @guliye91Posted almost 4 years ago

  You are better although your site is not responsive. Make use of css grids and flexbox for better layout. This is the same project you built. I built using css grid and flexbox. Kindly go through to get better understanding on how layout and responsivenes work https://www.frontendmentor.io/solutions/single-price-grid-component-using-css-grid-and-flexbox-RF2eD1iIC

  2

  ---

  

  Tyler Case•90

  @tcase629Posted almost 4 years ago

  Thank You.

  0