-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Deactivate sidebar for release notes #27800
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
Conversation
The release note page is only a structured collection of links to sub-pages. The sidebar would repeat all the titles of the sub-pages and thus basically repeat all the content of the page. Even worse, it does not contain the version sectioning structure of the release notes page itself. To keep the layout the sidebar gets one empty dummy section. If we leave the sidebar completely empty, it will be deactivated and change the layout of the page. Related to matplotlib#27690 (comment)
a88de27
to
abe4479
Compare
Agreed, the layout change is undesireable. We can work around that by an empty sidebar -> updated this PR. I would alternatively fine with the restructuring. Note however, that the links on the sidebar (e.g. "Version 3.8.0") would then only go to the index page of that version and you need to click a second link to get to some content. So the sidebar links for the restructured content are not too practical. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems good. Could put the front page sidebar in there, but this is fine.
I rather leave an empty sidebar. The front page sidebar has cheatsheets and the donate button, which are out of scope in the changelog context. |
The release note page is only a structured collection of links to sub-pages. The sidebar would repeat all the titles of the sub-pages and thus basically repeat all the content of the page. Even worse, it does not contain the version sectioning structure of the release notes page itself.
It's thus best to deactivate the sidebar here.
Related to #27690 (comment)