All comments

• 

  Elias Derbew• 130

  @EliasDerbew

  Submitted 7 days ago

  

  product overview

  1

  20

  

  KapteynUniverse• 100

  @KapteynUniverse

  Posted about 14 hours ago

  Hey, you can delete

  margin: 10px auto;
  max-width: 1440px;   
  gap: 1rem; */     
  

  from body and change height to min-height: 100vh;. It will do the same as what you did.

  Using max and min for height and widths would be better for responsiveness. Also rem and em units instead of pixels. These videos explain them great video1 video2

  0
• 

  Suzunatsu• 20

  @st0272

  Submitted 1 day ago

  

  Blog preview card solution

  1

  10

  

  KapteynUniverse• 100

  @KapteynUniverse

  Posted 1 day ago

  Nice work. I recommend you to use em and rem units instead of pixels. This video is a great explainer

  0
• 

  Patrick OUAMBO• 40

  @ouambo-takouam

  Submitted 1 day ago

  What are you most proud of, and what would you do differently next time?

  I have been able to center the page content.

  What challenges did you encounter, and how did you overcome them?

  I didn't know how to adapt the page for mobile screen. So may be my code won't display good on mobile.

  What specific areas of your project would you like help with?

  Mobile first

  

  Responsive Page using CSS Grid

  3

  11

  

  KapteynUniverse• 100

  @KapteynUniverse

  Posted 1 day ago

  Looks good. You might wanna use min-height, max-width instead of height and width and em-rem units for margins, paddings and text sizes etc.

  0