testimonials-grid-section

Community feedback

• 

  Lucas 👾•104,160

  @correlucas

  Posted almost 3 years ago

  👾Hello @tltd0807, congratulations for your new solution!

  Your solution is great, but the desktop version isn't properly aligned vertically. To make this alignment, first of all put min-height: 100vh to the body to make the body display 100% of the viewport height (this makes the container align to the height size thats now 100% of the screen height) size and display: flex e flex-direction: column to align the child element (the container) vertically using the body as

  The html markup is important and you can improve it using meaningful tags and replace the divs, for example the main div that takes all the content can be wrapped with <main> or section, about the cards you can replace the <div> that wraps each card with <article> you can wrap the paragraph with the quote with the 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.

  SEMANTIC TAGS: Here's a complete guide for HTML semantic TAGS: https://www.w3schools.com/TAgs/default.asp

  ✌️ I hope this helps you and happy coding!

  01

  Marked as helpful
• 

  David•7,960

  @DavidMorgade

  Posted almost 3 years ago

  Hello To Dat, congratulations on getting the challenge done, the layout and the responsiveness looks pretty good, nice job!

  If you don't mind I would like to give you a little suggestion:

  Try centering all your columns in the middle of the screen using flex-box, you just have to give a min-height: 100vh to the body so it wrapps the browsers screen, you should try something like this:

  body {
    font-family: "Barlow Semi Condensed", sans-serif;
    font-size: 13px;
    background-color: hsl(210, 46%, 95%);
    display: flex;
    justify-content: center;
    align-items: center;
    flex-direction: column;
    min-height: 100vh;
  }
  

  With this, everything will get completely centered!

  Apart from that, try using a bit more semantic tags on your html, for example, your container could fit perfectly on a <main> tag, since thats the main part of your html page.

  The rest of your CSS is fine in my opinion!

  Hope my feedback helps you, if you have any questions don't hesitate to ask

  01

  Marked as helpful