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

agogo theme: ensure hidden toctree is show on Table of Contents #6679

Merged
merged 1 commit into from Jun 16, 2022

Conversation

carandraug
Copy link
Contributor

The hidden option for the toctree directive is meant to hide from the
document at the location of the directive. This for when the links
are to be inserted in the HTML sidebar. However, since version 1.2,
the toctree() template function requires the includehidden=True
option.


Using the agogo theme, if a document has the toctree set to hidden (because one wants to have it displayed on the sidebar instead of inserted in the document), then the Table of Contents on the side bar will be empty. This fixes it by setting includehidden=True when calling toctree() template function. This is required since sphinx 1.2.

I am not sure if the same should be done to sphinx/themes/basic/globaltoc.html

The hidden option for the toctree directive is meant to hide from the
document at the location of the directive.  This for when the links
are to be inserted in the HTML sidebar.  However, since version 1.2,
the toctree() template function requires the includehidden=True
option.
@AA-Turner AA-Turner closed this May 23, 2022
@carandraug
Copy link
Contributor Author

Hi. This PR was closed but seems that the issue is not fixed yet. Did I miss something?

@AA-Turner
Copy link
Member

I meant to label this PR rather than close it -- thanks for catching. I've gone through the others I triaged & seen no other similar errors.

A

@AA-Turner AA-Turner reopened this May 23, 2022
@AA-Turner AA-Turner added this to the some future version milestone May 23, 2022
@JulienPalard
Copy link
Contributor

CI failing due to psf/requests#6140

@AA-Turner AA-Turner changed the base branch from master to 5.x June 16, 2022 20:09
@AA-Turner AA-Turner modified the milestones: some future version, 5.1.0 Jun 16, 2022
@AA-Turner AA-Turner merged commit 715a137 into sphinx-doc:5.x Jun 16, 2022
AA-Turner added a commit that referenced this pull request Jun 16, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants