Four-card-feature-section

Community feedback

• 

  Ye-Won Seo•170

  @yaywonah

  Posted 5 months ago

  • Your HTML and CSS were easy to read and follow along.
  • I would suggest changing the px values of your margin, width, padding, and font sizes to em/rem, so that the layout is more responsive.
  • The card divs in your mobile media query touch the edges of the screen once it hits less than 400px. I would change the

  .col1, .TeamBuilder, .Karma, .col3 {
  width: 100%
  }
  

  to a smaller value like width: 80%

  • I'd also change the margins of your h2 and h1 elements in your title div so that there's less space between them. That way it can look more like the design.

  00