Responsive Testimonial Grid Section with SCSS

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted over 2 years ago

  👾Hello Alexander Charles , congratulations for your new solution!

  Your solution is really good done, also the grid layout made with grid areas is amazing too. But you need to fix some things to improve it:

  The container is too big, you need to give max-width: 1110px to avoid it growing too much:

  .grid-container {
      position: relative;
      display: grid;
      grid-template-columns: repeat(4, 1fr);
      max-width: 1110px;
      gap: 2em;
      grid-template-areas:
          "one one two three"
          "four five five three";
  }
  

  You can improve the semantics replacing the card divs with <article> and the paragraph with the quote with the property tag <blockquote> this way you'll wrap each block of element with the best tag in this situation. Pay attention that <div> is only a block element without meaning.

  👋 I hope this helps you and happy coding!

  Marked as helpful

  1

  ---

  

  Alexander Charles Bonney•530

  @alexanderbonneyPosted over 2 years ago

  @correlucas Thanks for the recommendations. Lucas. They are really helpful.

  1