Social Proof Section With Flexbox

Community feedback

• 

  Lucas 👾•104,400

  @correlucasPosted about 2 years ago

  👾Hello Bassey Ukeme, Congratulations on completing this challenge!

  I’ve few suggestions for you that you can consider adding to your code:

  Fix the alignment. The best way make this alignment is by using FLEXBOX. The first step, is to add min-height: 100vh to make the body height size becomes 100% of the viewport height, this way you make sure the container will be aligned vertically since the body will display ever 100% of the screen height. After that add two flex properties to make the alignment display: flex / align-items: center; / justify-content: center;. If you're using the attribution you need to add flex-direction: column to make the attribution stays under the QR Code component. See the code below:

  body {
      min-height: 100vh;
      background: url(images/bg-pattern-top-desktop.svg) no-repeat fixed, url(images/bg-pattern-bottom-desktop.svg) no-repeat bottom right fixed;
      background-size: 500px 300px, 900px 500px;
      font-family: 'League Spartan', sans-serif;
      display: flex;
      align-items: center;
      justify-content: center;
  }
  

  The html markup for this solution its built entirely with <div>and you can improve it using semantics tags to describe what its inside each block of content. 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.

  ✌️ I hope this helps you and happy coding!

  Marked as helpful

  0

  ---

  

  Bassey Ukeme•210

  @bukemePosted about 2 years ago

  @correlucas thanks a lot

  0
• 

  Hyron•5,870

  @hyrongennikePosted about 2 years ago

  Hi @bukeme,

  Nice job on the challenge

  Instead of using margin to try center the card you can replace your main rule with the following.

  main {
      max-width: 80%;
      margin: 0 auto;
      display: flex;
      flex-direction: column;
      justify-content: center;
      align-items: center;
      min-height: 100vh;
  }
  

  Hope this is helpful

  Marked as helpful

  0

  ---

  

  Bassey Ukeme•210

  @bukemePosted about 2 years ago

  @hyrongennike thanks a lot

  0