Landing page with flex-boxes

Community feedback

• 

  Ren Ras•420

  @renrasPosted almost 3 years ago

  There are a lot of things to improve on this project but here's a few to start with.

  Just nest the margin and padding inside the * selector also since it applies to all html element.

  *,
  *::before,
  *::after {
      font-family: 'Poppins', sans-serif;
      box-sizing: border-box;
  + margin: 0;
  + padding: 0;
  }
  

  Your body has a max width making your page not extending on larger screens. You can apply this to the elements inside instead.

  body {
      color: black;
    - max-width: 1440px;
  }
  

  Your images are overflowing because they have a fixed width. What you can do is to apply properties like width:100%, max-width and min-width instead.

  img {
   + width: 100%,
   + height: auto,
  }
  

  You can apply this concept on your containers too.

  You can change the color of the svg directly on the file by changing it's fill attribute. If I want two of them but different color I just make another copy.

  Goodluck!

  Marked as helpful

  0