-
Notifications
You must be signed in to change notification settings - Fork 321
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
bd-article-container
is not taking full width of its parent
#904
Comments
this started when we started implementing the ng-sphinx structure in the theme (#861). To be fair, I don't really understand why people want this fix 80-100 fix character per page and enforce it. I use huge screens at home, I'm simply never displaying anything full screen and I deal myself with the width I want to see ;-) On the other hand, I can confirm the bug from one of my documentation as well in pages autogenerated by |
Adding html_sidebars = {
"**": ["search-field.html", "sidebar-nav-bs.html"]
} in my .bd-article-container {
width: 100%;
} in my |
In your modification you are adding back some widgets in the left sidebar which is not necessary in my case.
|
I think this is a page in our documentation that shows this issue (not as pronounced because the content is wider but still there for debugging purposes): https://pydata-sphinx-theme.readthedocs.io/en/stable/examples/subpages/subsubpages/index.html I think the re: limiting the word count per line, I don't have strong opinions other than we should follow best-practices that are generally recommended in industry / do what professional-looking documentation engines/sites do. I'm not a designer so I try to limit my personal design opinions in favor of what more well-resourced projects do 🤷 |
@bloussou: applying this css change solves the issue for matplotlib (see matplotlib/matplotlib#23976) partially, but it creates a big white top margin: |
Hello,
Moving from
0.9.0
to0.10.1
It looks like.bd-article-container
as amax-width: 60em;
set. It doesn't look great in my opinion:In
0.9.0
.bd-article
in.bd-content
was taking full width with a much nicer result :The change was introduced by this PR : 4b97fe0#diff-49f6c89aa34e046c7b8d0f3776503de95dd1befffe5f2e13d032c136264ec6cf
The text was updated successfully, but these errors were encountered: