Huddle landing page with alternating feature blocks

Community feedback

• 

  Abhilashpandey•270

  @freakyjonesPosted about 2 years ago

  Hello Ali, congratulation on finishing your challenge.

  I just saw your code and I would like to give some suggestions with your permission,

  1. keep your images in one folder. it will help you manage your project in long run.
  2. use a universal selector (*) to get rid of default browser styles like *{margin:0,padding:0,box-sizing:border-box}.

  1

  ---

  

  Ali Mousavi•60

  @Ali-PrimerPosted about 2 years ago

  @freakyjones oh really thanks man, I will use it in my other projects but about your first suggestion I have to say I don't know how to work with github very well, then I have to do this GOOD LUCK (;

  1
• 

  David•8,000

  @DavidMorgadePosted about 2 years ago

  Hello Ali congrats on finishing the challenge!

  Your page looks good on desktop sizes but doesn't have adaptation for lower device, I recommend you to start working on it, you will learn a lot from CSS when you start making your websites responsive using mediaqueries, flexbox, css grid... !

  You could even try the mobile first approach, start building from mobile and then adapt it to larger sizes like tablets and desktop devices!

  Hope my feedback helps you!

  1

  ---

  

  Ali Mousavi•60

  @Ali-PrimerPosted about 2 years ago

  @DavidMorgade Thanks David I have to work on my self in it

  0