RESPOSIVE LANDING PAGE USING CSS FLEXBOX

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted over 2 years ago

  👾Hello Ruchi, congratulations for your new solution!

  Hey Ruchi, your solution seems good, I can see that you've done a good work here, if you fix the main section alignment and the icons will be even better. See my tips below:

  To have a good size for the icons (they're too big) use a standard value as 32px or 24px. In this case is better 32px:

  .icon {
      /* width: 1.5rem; */
      max-width: 32px;
      border: 1px solid rgb(0, 0, 0);
      padding: 1.5%;
      color: white;
      border-radius: 100%;
      margin: 2%;
  }
  

  Removing the paddings/margins you add to align the section and instead use display: flex to align and min-heigh: 100vh to make the body has 100vh as its size to align the child vertically

  body {
      display: flex;
      min-height: 100vh;
      background: url(images/bg-mobile.svg) no-repeat 50% 0%,hsl(257, 40%, 49%);
      background-size: 100%;
      margin: 10%;
      animation-fill-mode: backwards;
      flex-direction: column;
      align-items: center;
      justify-content: center;
  }
  

  👋 I hope this helps you and happy coding!

  Marked as helpful

  0