All comments

• 

  nelson•160

  @HyoganSubmitted about 2 years ago

  -The overlay was hard. But i thing i managed to do the job. -i'm happy of the rotation animation on the stats, i think they were my first -Please look at my code , and tell me what i made wrong

  

  my eternal css flex, always with me

  1

  30
  

  Virgile•150

  @virgile-labPosted about 2 years ago

  Hi Nelson !

  I like the rotation animation !

  Here a suggestion :

  • To center the card, add in the body tag min-height: 100vh;, and delete in the .main-bloc class margin-top: 200px;

  Hope it will help you !

  0

  This comment has received 1 replyShow
• 

  Fadi•40

  @fadisayeghSubmitted about 2 years ago

  Newbie here, starting my coding journey.

  I would appreciate any suggestions, comments, tips, and advice.

  

  Basic

  1

  20
  

  Virgile•150

  @virgile-labPosted about 2 years ago

  Hi Fadi !

  Here are some tips :

  1. It is more suitable to use the <main> tag instead of <div class="main">
  2. <btn> tag is not a part of HTML5 semantic, you should use instead <button>
  3. Your layout is not responsive, so to do it, you can use display: flexbox or display: grid. You will find more information here
  4. To center the card, you could use Grid in the class .main eg. :

  .main {
  display: grid
  place-items: center
  min-width: 100vw;
  min-height: 100vh;
  }
  

  Hope it will help you !

  0