Meet landing page

Community feedback

• 

  P

  Rohloffmeister•300

  @RohloffmeisterPosted about 2 months ago

  Hi, Your Website looks good so far. To start with your problem you could try putting the footer outside of your <main> (this also helps with accessibility. If you want to know more google "HTML Semantic Elements"). I also saw that the 4 images can get pretty big. I personally would use the grid property.

  /* Dekstop and Tablet*/
  .img-grid{
  display: grid;
  grid-template-columns: repeat(4, 1fr);
  }
  .img-grid img{
    width: 15wv;
  }
  
  /* Tablet  */
  
  .img-grid{
    display: grid;
    grid-template-columns: repeat(4, 1fr);
  }
  .img-grid img{
    width: 22wv;
  }
  
  /* Mobile */
    .img-grid {
      display: grid;
      grid-template: repeat(2, 1fr) / repeat(2, 1fr);
    }
  
  .img-grid img{
    width:40wv;
  }
  
  

  Thats how i did it. I hope this helped you a bit.

  Marked as helpful

  0

  ---

  

  P

  yele.m•330

  @YelemyahMPosted about 2 months ago

  @Rohloffmeister Thanks for the feedback. Great idea to set footer outside.

  1