Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Huddle landing page with alternating feature blocks

Ali Mousavi• 60

@Ali-Primer


Design comparison


SolutionDesign

Solution retrospective


comment what you think about it

Community feedback

Abhilashpandey• 270

@freakyjones

Posted

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-Primer

Posted

@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

@DavidMorgade

Posted

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-Primer

Posted

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

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord