3-column preview card component

Community feedback

• 

  P

  tediko•6,680

  @tedikoPosted over 3 years ago

  Hello, Pavan! 👋

  Great work! What i would suggest is to remove div with class .flex and move these styles to .container div. We don't need two divs, cleaner code. Then I would suggest to add some padding on top/bottom of body/container because your container stick to top/bottom on mobile. Lastly, you should create common class for your cards, so you don't repeat code - in your case you repeat only padding, but in future projects it may be more styles. So you should have f.e .card .card1' in classes where .card` contains the same styles for each box.

  Good luck with that, have fun coding! 💪

  0
• 

  Valeriu•150

  @valy-sPosted over 3 years ago

  Hi Pavan,

  I think one way to optimize your code is to try writing the code for mobile first, think you should minimize the your CSS code.

  0