All comments

• 

  Ngunza Francisco• 90

  @NgunzaFrancisco

  Submitted 10 months ago

  

  3 Column Preview Card Component

  1

  10

  

  Aleksandr Bagaev• 50

  @sobbaka

  Posted 10 months ago

  • Background and cards size differ from design
  • I recommend to create :root in CSS and use variables from there (for example for colors - makes life easier)
  • If you start to use em - try to use it everythere
  • add white border into links (border: 1px solid #fff;) - without it bottom of card 'jumping' on hover
  • and about class naming of cards. I recommend to use content__card card card--sedan. content__card to place card in content, card is to make it independent reuseable element
  • use text-transform: uppercase; instead capslock in H1

  0