Responsive grid layout page

Community feedback

• 

  Lucas 👾•104,220

  @correlucasPosted over 2 years ago

  👾Hello @alseny-diallo, congratulations on your new solution!

  I’ve some suggestions for you:

  1.Fix the alignment of the whole content using flex and min-height to manage the vertical alignment and make everything centered.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 that's 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 reference.

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

  2.You’re in the right track I can see that you’ve used the majority semantic tags possible for this challenge, the only block you’ve missed is the paragraph containing the quote text you can improve the accessibility there using <blockquote> to indicate to screen readers that the content inside that paragraph is a quote.

  ✌️ I hope this helps you and happy coding!

  Marked as helpful

  0

  ---

  

  Alseny•50

  @alseny-dialloPosted over 2 years ago

  hi @correlucas okey thanks

  1