HTML5 and CSS with SASS to compile the CSS.

Community feedback

• 

  Mojtaba Mosavi•3,760

  @MojtabaMosaviPosted over 3 years ago

  Well done, The fontsizes are a litle too big on the tablet view.

  Keep coding :=)

  1

  ---

  

  Flamick•65

  @FlamickPosted over 3 years ago

  @MojtabaMosavi Yeah I thought it was kind of big but I just didn't like all the empty space. I'll lower it a bit though. Thanks for the input. :)

  0
• 

  Dušan Lukić•1,660

  @dusanlukic404Posted over 3 years ago

  Hi Flamick, well done!

  • Add transition to buttons
  • Center both horizontally and vertically main card
  • Headings should be smaller on a mobile view

  0

  ---

  

  Flamick•65

  @FlamickPosted over 3 years ago

  @dusanlukic404 I added a transition for smoother animation. Also, I'm not sure what you mean by centering the main card horizontally and vertically.

  0

  

  Dušan Lukić•1,660

  @dusanlukic404Posted over 3 years ago

  @Flamick if you add hover state you just changed the state of that element when you're hovering over it with cursor. Transition is something like animation, so I suggest you to set this and you will see the difference: .btn { transition: all 300ms ease-in-out; }

  • For centering that card you should avoid that vertical scrollbar. Use overflow: hidden; on body

  Marked as helpful

  0

  

  Flamick•65

  @FlamickPosted over 3 years ago

  @dusanlukic404 I get what you mean by that now. I'll have to give that a try whenever I hop on next. Thank you.

  0

  

  Dušan Lukić•1,660

  @dusanlukic404Posted over 3 years ago

  you can try it now to see. Go on inspect and then add this style to buttons.

  0