html and css, flexbox and media queries

Community feedback

• 

  Antonio Novak•480

  @Nova988Posted about 3 years ago

  Hey Barath, I have nothing more to add then what Dusan already said except 1 tip: You can add border-radius to the card class instead of the car classes you did. It makes the code a bit easier I think.

  Marked as helpful

  0

  ---

  

  Bharath Shet•65

  @Bharath-ShetPosted about 3 years ago

  @Nova988 thank you

  0
• 

  Dušan Lukić•1,660

  @dusanlukic404Posted about 3 years ago

  Hey Barath! Good work but I have some suggestions for you:

  • You replaced position of SUVs and Sedans. Sedans should be on first card
  • Add more padding to get more white space between content and cards
  • Add outline 2px solid #fff instead of border on btn:hover because borders in this case makes cards bigger on hover effect cause they take up some space (Or just add border witdth and style and make color be transparent) <- that's better solution
  • Make buttons bigger

  Marked as helpful

  0

  ---

  

  Bharath Shet•65

  @Bharath-ShetPosted about 3 years ago

  @dusanlukic404 thank you

  0