All comments

• 

  Jonathan•105

  @JonMStevensSubmitted about 3 years ago

  I did this challenge to try and get more comfortable with CSS grid. Any feedback is appreciated, and feedback about grid is particularly welcome.

  

  3 Column Preview Card Component

  3

  10
  

  Erick F. Felix•265

  @efs0-cod3Posted about 3 years ago

  Your code looks great you should check some things like border radius and their position and the position of the attribution Instead of div I would change it to footer. Otherwise is looking good minor things. Hope it helps.

  Marked as helpful

  1
• 

  Ramiro Solano•35

  @ramirishoSubmitted about 3 years ago

  Second project that I do, I will appreciate any advice, criticism and recommendation :D

  

  CSS flexbox and BEM methodology

  2

  10
  

  Erick F. Felix•265

  @efs0-cod3Posted about 3 years ago

  For the html issue you should have you div’s inside a main tag (landmark).

  Looking pretty good on desktop view.

  0

  This comment has received 1 replyShow
• 

  Artem Marchenko•50

  @ArtemVMVSubmitted about 3 years ago

  Hey everyone! Just starting, hope that's vital =D

  

  HTML,CSS, responsive design

  2

  20
  

  Erick F. Felix•265

  @efs0-cod3Posted about 3 years ago

  Good solution Artem. I would remove the line-height and font-weight on .article for more aesthetics.

  good luck!

  0
• 

  Abhishek Verma•15

  @abhishekanimatronSubmitted about 3 years ago

  I struggled a bit to make it responsive, if anyone can look at the codebase and give me ideas as to how i make the card responsive would be super helpful! Thanks in advance! Also thanks frontend mentor!

  

  HTML, CSS

  2

  10
  

  Erick F. Felix•265

  @efs0-cod3Posted about 3 years ago

  -Try to change the #outer-container size from % to px. -Check the background too!

  0