Technical Documentation Suggestion

This post has been deleted.

1 Like

hi @mackenzie302

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 single li element. why don’t just nested with single div and get rid of the ul and li.

regards

here read this

1 Like

and i implemented it to my tech docs.

https://codepen.io/sobadrdb/pen/GRoKQQw

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