Huddle landing page with alternating feature blocks
Design comparison
Solution retrospective
comment what you think about it
Community feedback
- @freakyjonesPosted over 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,
- keep your images in one folder. it will help you manage your project in long run.
- use a universal selector (*) to get rid of default browser styles like
*{margin:0,padding:0,box-sizing:border-box}
.
1@Ali-PrimerPosted over 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 - @DavidMorgadePosted over 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-PrimerPosted over 2 years ago@DavidMorgade Thanks David I have to work on my self in it
0
Please log in to post a comment
Log in with GitHubJoin 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