@FluffyKas
Posted
Heyo,
I'm not working as a professional yet but I'm actively looking for jobs and sass/scss is something I see very frequently in job requirements. It doesn't take long to learn and you just use as many of its features as you wish.
Your solution looks great btw, I only have some small suggestions really:
-
Don't forget to set the font-family of your button. If you don't set it or don't reset it somehow, it's just going to use the browser's default font.
-
Careful with your heading levels. It's better to have only one <h1> heading, and work your way down from there to create a clear document outline (much like with a book that has a main title and chapter titles, etc).
-
In the why us section you actually want to use a <ul> witl an <li> for each item and then style it with css. Don't use <br> for this, that has a very specific purpose.
Marked as helpful
@Al-lien
Posted
Hello @FluffyKas ! Thanks a lot for giving me your feedback, i really appreciate it.
How do you suggest I prioritize my code ? Do you think I should set "Join our community" as <h1> and "Why us" as <h3> ?
I'm going to learn sass then and I will soon update my solution with all your advice.
Have a great day !