-
Notifications
You must be signed in to change notification settings - Fork 6
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
docs: ignore panel css to increase main content width #104
Conversation
doc: ignore panel css to increase main content width Otherwise the main content ends up being weirdly small See: https://github.com/executablebooks/sphinx-book-theme/blob/547cc95674623345124d9bbea9e15e5b5fa34d0e/docs/conf.py#L62 https://sphinx-panels.readthedocs.io/en/latest/index.html?highlight=panels_add_bootstrap_css#sphinx-configuration
Hey, cool, thanks for looking into this :) I still have to think about this a bit though. I noticed that the pages of EPB projects all have this wider view, but I personally prefer to have texts less wide. It's for the same reason that newspapers (and many journals) write in columns: it's more convenient for reading. And if you prefer a wider view, you can collapse the left sidebar. (At the same time though, you could say readers should just make their window smaller if they prefer...) |
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
I've this discussed your suggestion a bit in my group. They are in favour of a wider page width, so I'll merge this one. Thanks again for noting this! To summarize:
|
doc: ignore panel css to increase main content width
Otherwise the main content ends up being weirdly small
See:
https://github.com/executablebooks/sphinx-book-theme/blob/547cc95674623345124d9bbea9e15e5b5fa34d0e/docs/conf.py#L62
https://sphinx-panels.readthedocs.io/en/latest/index.html?highlight=panels_add_bootstrap_css#sphinx-configuration
Old:
New: