@mattstuddert
Posted
Looks great Davide!
I noticed a few things while having a look through your code:
-
In your CSS you don't need to use the
*
selector for properties such asfont-size
,color
andfont-family
, as these properties can make use of CSS inheritance. Here's a great article about it if you want to read more. -
The
background-image
on thebody
could do with havingbackground-size: cover
added to it to make sure it stretches to fit the screen. -
Nice work using a
min-width
media query. This helps keep your loaded styles as lean as possible for mobile users. -
Quick heads up that you've also got a random closing
footer
tag in your HTML without an opening tag. -
Great use of HTML5 semantic elements as well 👍
Hope that helps. Keep up the great work! 🙂
@Banjer71
Posted
Thank you very much Matt. I will fix the things you mentioned. ;)