Responsive three layout box using flexbox

Community feedback

• 

  Muhammad Muzzammil•350

  @mk-muzzammilPosted about 1 year ago

  Yeah a recomendation for you I have seen your code. try to use local variables made it in :root{} and use it on the whole stylesheet. Another recomendation is you set buttons using position absolute try it to do with flex using align-self:baseline .if you are using flex property then try to position it using relative units like vh,% it is more helpful to you in position elements more accurately even in responsive desings .Main Fault i have found in your project see your responsive mobile preview because you set height 100vh cards are not showing correctly it will go beshind the screen try not to set the height in these cases. hope you will like some recomendations any other query let me know.

  Marked as helpful

  1

  ---

  

  Adetona Blessing•150

  @adetona54Posted about 1 year ago

  @MK-Muzzammil The reason why I didn't use align-self:baseline; is because, when I do, It kind of increase the height of the box. I don't know if I have made a mistake somewhere, but still, I appreciate the time you've taken to review my code.

  0
• 

  naroors•30

  @naroorsPosted about 1 year ago

  The car images are kinda off. Also add more padding to the buttons. The "title" is too far away from the paragraph text. And the buttons should have some more margin-bottom.

  But its still good.

  0