Four card feature section

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted over 4 years ago

  Awesome work, Inkz, and congrats on submitting your first solution to the platform! You've done a really good job matching it up to the design. Here are a couple of small pointers after taking a look at your code:

  • You've used a div for the .card-title elements, whereas I'd say they make more sense as headings. Seeing as your only other heading level is a h1 on this project, h2 headings would work well.
  • Avoid making your CSS selectors unnecessarily complex. For example, you've got .cards-container .card .card-image when you could just have .card-image. This will help keep your styles more maintainable. Especially as you build larger and larger projects.

  I hope this helps. Keep up the great work! 👍

  0

  ---

  

  Inkz•250

  @inkinkzPosted over 4 years ago

  @mattstuddert Thank you so much! This really helps me :)

  0

  

  T

  Matt Studdert•13,611

  @mattstuddertPosted over 4 years ago

  @inkinkz you're welcome! Happy to help 🙂

  0