Responsive landing page ❤️

Community feedback

• 

  Pulkit saxena•890

  @Pulkit-s21Posted over 2 years ago

  There are a lot of issues in the code.

  1. Color the background instead of making a box.
  2. There is no image in the mobile version of your code
  3. The button is in the middle of the box and not at the end. The components aren't spaced evenly and stick together in desktop version

  Marked as helpful

  1

  ---

  

  Jishan•290

  @jish0101Posted over 2 years ago

  @Pulkit-s21 Thanks, i tried to change it again, see if it looks better?

  0

  

  Pulkit saxena•890

  @Pulkit-s21Posted over 2 years ago

  @jish0101 The image is there in the mobile view as well now so kudos on that. Try to set the body min-height to 100vh so that the color is all the way through cause rn it falls shorts and set a max-width for the card in the mobile view like 35em or something like this before it changes to desktop view because it keeps on stretching to the size of the screen and doesnt look that nice. With a fixed max-width it wont span too much and look like a neat card

  1

  

  Jishan•290

  @jish0101Posted over 2 years ago

  @Pulkit-s21 done!

  0
• 

  Wangai•410

  @WangaiJMPosted over 2 years ago

  Apply the background color to the body rather than the outer-container or fill the body with the outer-container

  .outer-container{
     width: 100%;
     height: 100%
  }
  

  Marked as helpful

  0

  ---

  

  Jishan•290

  @jish0101Posted over 2 years ago

  @WangaiJM done! thanks😊

  0