Mobile first and using Flex

Community feedback

• 

  Priyanka•150

  @priyankaladPosted over 3 years ago

  Hey,

  I have gone through your code, and it's pretty much cool except few alignments. I would recommend to implement this challenge using flexbox.

  Also, I noticed that feedback card containers are currently left aligned in the mobile view. To make them middle aligned, you can add 'margin: 20px auto' to the class "cards-container".

  Let me know if you need any further help

  Happy Coding :)

  2

  ---

  

  kirubakk•245

  @KIRUBASHANKAR26Posted over 3 years ago

  @priyankalad Thank you so much

  0
• 

  Raymart Pamplona•16,100

  @pikapikamartPosted over 3 years ago

  Just a suggestion. Instead of adding margins in the flex child, yeah child. You could have a flex-basis or flex giving a width, then just add a gap property in the flex container so that the child doesn't scale via the margin used^

  1

  ---

  

  kirubakk•245

  @KIRUBASHANKAR26Posted over 3 years ago

  @pikamart Thank you

  0

  

  Raymart Pamplona•16,100

  @pikapikamartPosted over 3 years ago

  @KIRUBASHANKAR26 Your welcome^

  0