Insure Landing page using HTML and CSS

Solution retrospective
feedback will be highly appreciated thank you.
Please log in to post a comment
Log in with GitHubCommunity feedback
- @emestabillo
Hey @lesego16, congrats on submitting this project! Here are a few pointers:
-
I think you can merge your CSS into one file. Place the media queries at the bottom of the file.
-
Use
rem
s orem
s instead ofpx
-
Avoid using
id
s for styling. It has a high specificity and difficult to override if needed. Useclass
es instead. -
The links on the mobile menu shifts as you hover on them. I would add the border initially as transparent so it is already occupying space, and then change color when the user hovers.
-
The patterns are purely decorative. I would use pseudoelements to position them instead of including them in html inside
img
tags. -
The paths to some of the background images are incorrect and giving you an error in the console. I think they're missing the two leading periods
../
Hope this helps!
Marked as helpful -
- @lesego16
Thank you, I will try to correct it
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