Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Doc sidebar content overflow #791

Open
Helveg opened this issue Nov 15, 2023 · 3 comments · May be fixed by #813
Open

Doc sidebar content overflow #791

Helveg opened this issue Nov 15, 2023 · 3 comments · May be fixed by #813

Comments

@Helveg
Copy link

Helveg commented Nov 15, 2023

The documentation sidebar content flows over into the API docs here (on Chrome 119.0.6045.124, without any extensions): https://neuralensemble.org/docs/PyNN/reference/neuronmodels.html

@sdivyanshu90
Copy link

Hi,
I noticed this issue and would love to work on fixing it. Could you please provide any additional details or specific requirements, if needed?
Thanks.

@apdavison
Copy link
Member

Hi @sdivyanshu90 - we would be pleased to have someone fix this. The documentation is built with Sphinx, the source is in the doc directory.

One way to fix it might be to reduce the number of levels, e.g., we only need to go as far as "IF_cond_exp", the next level ("IF_cond_exp.injectable") is not needed.

I'd also be ok with changing the theme (currently "nature"), if you can find a more modern theme that fixes the sidebar problem.

@sdivyanshu90
Copy link

Hi @apdavison,

Thanks for the details! I'll start working on this and explore both approaches - reducing the sidebar levels and trying out a more modern Sphinx theme to resolve the issue. I'll submit a PR soon with the fix.

Let me know if there's anything specific you'd like me to consider while making these changes.

Thanks!

@sdivyanshu90 sdivyanshu90 linked a pull request Feb 28, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants