Testimonials grid section with CSS GRID

Community feedback

• Account deleted

  Hi,

  Your solution looks ok, but here's some things I think you should fix;

  • You must have a main tag in every challenge that you do to specify the main content of the document.
  • Your container seems to be a big bigger compared to the design but it doesn't matter since we not given dimensions, though I'd recommend decreases them a little bit.
  • You seem to have the wrong color on the paragraphs, yours is bit lighter and hard to read.
  • I think you must also do a layout for tablets to have a smoother transition to mobile, and to avoid having a huge empty space on the bottom of some containers when you resize.

  Keep coding👍.

  Marked as helpful

  1

  ---

  

  Gian Franco•290

  @gianbackPosted about 3 years ago

  @thulanigamtee thanks for your words! As it was a challenge that I had saved, I forgot to change that div to the main, in the subject of the letter of the paragraphs I put a lot of opacity I think haha, btw, what would you recommend so that the containers have a more fluid transition?

  0

  Account deleted

  @gianback If you'd like to keep it this way it's totally fine but making a layout for tablets would make it a whole lot better based on the responsiveness.

  1

  

  Gian Franco•290

  @gianbackPosted about 3 years ago

  @thulanigamtee thank you, i will try tomorrow, your words help me a lot

  0