remix-run / remix-website

322 stars 72 forks source link

fix: account for menu categories that are also links #240

Closed brookslybrand closed 4 months ago

brookslybrand commented 4 months ago

This thing is pretty ridiculous

The way our docs works you can technically have a category menu be a page itself, in which case it needs to be a link.

Additionally that doc can have children, so it still needs to be a collapsable menu. This is a monstrosity, but no idea if we ever did this with previous releases, so need to give a decent default behavior.

I did my best to clean up the code and colocate css that should be the same.

Also, sorry for my juvenile test doc names

https://github.com/remix-run/remix-website/assets/12396812/4a442310-1720-4486-a17b-cf08830acefa