Desktop Landing page alone

Community feedback

• 

  MikDra1•6,920

  @MikDra1Posted about 2 months ago

  I encourage you to use this technique to make the card responsive with ease:

  .card {
  width: 90%;
  max-width: 37.5rem;
  }
  

  On the smaller screens card will be 90% of the parent (here body), but as soon as the card will be 37.5rem (600px) it will lock with this size.

  Also to put the card in the center I advise you to use this code snippet:

  .container {
  display: grid;
  place-items: center;
  }
  

  Hope you found this comment helpful 💗💗💗

  Good job and keep going 😁😊😉

  0
• 

  Djamel1133•620

  @Djamel1133Posted about 2 months ago

  Hi, well done... I am not expert but i think it is better to removed duplicate height: 100% from html, body since height: 100vh is already set in body and remove some unused properties like width: 100% in main (it is unnecessary since it will naturally take the full width of its parent). you can (for this chalenge) justify your content like this : main{ height: 100vh; align-content: center; /* just this to center your content ...... } Again great job ...keep trying.

  0
• 

  GonzaSalda•30

  @GonzaSaldaPosted about 2 months ago

  I liked how you organized your HTML and CSS code. Everything is well structured and easy to read.

  0