PuRe CSS and Grids 😃

Community feedback

• 

  Dušan Lukić•1,660

  @dusanlukic404Posted about 3 years ago

  Hey Rami, well done! You did very good job.

  • Set display flex to body and also height of 100vh so you can easily center your main card
  • Then set overflow to hidden so you won't see vertical scrollbar
  • Make button more visible with box-shadow
  • Don't use <h2> instead of <h2>. You should think about it semantically. <h1> should come before <h2> in a document. <h2> is a sub element of <h1>.

  Marked as helpful

  1
• 

  P

  Patrick•14,265

  @palgrammingPosted about 3 years ago

  Looks good you are just missing your shadow around your layout

  Marked as helpful

  0

  ---

  

  Ramy Campusano Volquez•340

  @Daniels-notPosted about 3 years ago

  @palgramming Thanks for your feedback 🔥.

  0
• 

  Ahmed Faisal•5,095

  @afrusselPosted about 3 years ago

  Your main container is not center align both vertical and horizontal. Use display flex, align-item center and others

  1

  ---

  

  Ramy Campusano Volquez•340

  @Daniels-notPosted about 3 years ago

  @afrussel Thanks for your feedback 🐻.

  0