This post has been deleted.
1 Like
good job, the page is responsive.
i just want to say:
- you’ve got
.navbar
class duplicated in your css. - tbh, it was the first time i noticed someone nested all the
a
element in singleli
element. why don’t just nested with singlediv
and get rid of theul
andli
.
regards
here read this
1 Like
It looks good on mobile. My only suggestion would be a floating “Back to Top” butting since there’s a lot to scroll through.
Also the Back to Top link you have doesn’t take the user all the way to the top. It stops shy of the Nav Bar. Not sure if that’s what you wanted or not but with it this way I have to scroll up to get the NavBar visible. You can use an empty <div>
at the very top of your html and anchor your “Back to Top” link to that.
Here’s how I solved it on my page. https://codepen.io/jfirestorm44/full/oNbqmzB