product-preview-card-component-main

Community feedback

• 

  BENGUELLA Mohammed Elamien Tahar•30

  @thrbngPosted about 2 years ago

  Hi @Georgeinnerside , Congrats on your 2nd challenge

  first of all u have a big problem with colors , so try to use the file "style-guide" to get the correct colors , and also get the correct font-family and font-width. Then u need to add a height to the body like "`100hv".

  Learn more about CSS and always keep learning

  thr

  Marked as helpful

  0

  ---

  

  Georgeinnerside•80

  @GeorgeinnersidePosted about 2 years ago

  @thrbng Thank you so much, I will work on that.

  1
• 

  Patrick•800

  @PPechmannPosted about 2 years ago

  Hey George!

  Congrats on your 2nd challenge, very well done!

  I checked your code and saw that you centered the card with flexbox correct.

  You only need to add a height to the body, in this case of 100vh to cover the full screen, This will perfectly center the card, if you get rid of the 15% margin.

  With Flexbox, the styles align-items: center and justify-content: center, work when a height is applied to the parent element.

  Hope this helps and looking forward to see more solutions from you.

  Happy weekend and happy coding :)

  Patrick

  Marked as helpful

  0

  ---

  

  Georgeinnerside•80

  @GeorgeinnersidePosted about 2 years ago

  @PPechmann Thank you so much, I will work on. the corrections.

  1