@pikapikamart
Posted
Hey, good work on this solution.
-
Upon looking at it, there seems to be an unwanted scrollbar. I looked into the css and found out that one causing it is your
media-copyright
selector inside yourfooter
element. Inside themedia-copyright
the firstdiv
element nested in it have a declaration ofwidth: 120%;
remove that and you will fix the layout. -
Also while you're at it, adding
cursor: pointer
to the different button elements will be awesome, because adding it makes the element more natural since it is an interactive element. -
Making the breakpoint a lot sooner. As I resize it, the first section, the text gets very squished and another scrollbar appears at the bottom. So making a lot sooner for your breakpoint will be really good so that you could avoid those.
But reaching the mobile view, the layout is good and nothing different in it.
Those mentioned above will be really awesome if you could address it. Still, good work on this one^
@under4
Posted
@pikamart I cannot believe i missed that scrollbar! Thank you for bringing it to my attention. Awesome feedback!!