Simple 3 Column card Component using BEM CSS Format

Community feedback

• 

  nikos andriopoulos•0

  @nikosAndr13Posted almost 3 years ago

  Some things I would like to point out:

  1. A good practice for your HTML is to make everything classes or id just so you can focus and increase the specificity
  2. Now you had only a few images so it wasn't a problem, but when you put let's say 25 imgs in a folder you need to make a half/full step: The syntax: <img src="./images/icons/one-image.png">
  3. For the mobile responsiveness I would recommend finding the most common screen sizes and stick to that: for PC 1200px, iPad 768px iPhone 320px etc I hope I helped! Keep grinding!

  1
• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello! Great job on this one! Your solution is pretty much the same to the original design. The border-radius looks fine during the transition to smaller devices. Nothing much to say but I could suggest the following for you:

  • Adjust the height on the .container to make it appear longer.
  • Add a padding or margin around the paragraph inside the .attribution in order to give it enough space on smaller screen view.

  These are just very minor enhancement. Overall the project you made is on point. Keep it up!

  1