Dharmik_487• 1,740
@Dharmik48
Posted
Hey👋,
Your solution is really good! But.. I found a few suggestions:
- Try to use semantic html tags like
main
,section
,header
, etc. more as it is a good practice and good for SEO. - Also, I think you should remove the
margin-top: 10rem;
from886px
media-query as because of it there is a vertical scrollbar.
Keep it Up👍
Marked as helpful
0
Andrew• 260
@andy-devs
Posted
@Dharmik48 Thanks! Appreciate your advice
0