oceanhackweek / oceanhackweek.github.io

GitHub repo for the OceanHackweek website
https://oceanhackweek.org/
12 stars 38 forks source link

Rebuild OHW22 toctree #246

Closed abkfenris closed 1 year ago

abkfenris commented 1 year ago

It looks like a page needs to be in an existing toctree, before Sphinx will build a toctree for it.

I've added it back to the primary toctree, but I've limited the number of entries that will display in the menu.

I've also gone and removed the duplicate search options from the menu while I was there.

Closes #245

netlify[bot] commented 1 year ago

Deploy Preview for oceanhackweek-preview ready!

Name Link
Latest commit 22c930bc74a1bf6a9def4c083fc9e9646e83a5a3
Latest deploy log https://app.netlify.com/sites/oceanhackweek-preview/deploys/646e46e88bf2430008543aad
Deploy Preview https://deploy-preview-246--oceanhackweek-preview.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

emiliom commented 1 year ago

It looks like a page needs to be in an existing toctree, before Sphinx will build a toctree for it.

I've added it back to the primary toctree, but I've limited the number of entries that will display in the menu.

So, OHW22 is now in the top bar. It sounds like you intended to have header_links_before_dropdown effectively clip it, right? Maybe set header_links_before_dropdown to 2?

I've also gone and removed the duplicate search options from the menu while I was there.

Thank you! That's been a bit annoying. But, at least on the netlify preview, the search box is now gone.

Thanks!

abkfenris commented 1 year ago

I think netlify was ignoring the search and the limits as it was using an old version of the theme.

abkfenris commented 1 year ago

And it was using an old version of Python.

abkfenris commented 1 year ago

And it doesn't support 3.10. Grr.

abkfenris commented 1 year ago

Ok, now @emiliom it should be working.