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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix the docs contents menu #9529

Closed
wants to merge 1 commit into from
Closed

Fix the docs contents menu #9529

wants to merge 1 commit into from

Conversation

mattr-
Copy link
Member

@mattr- mattr- commented Jan 15, 2024

This is a 馃悰 bug fix.

Summary

Captures are static so we can't use one here otherwise the last page to
include the file is what wins due to the way include caching works
during generation.

Context

Closes #9527

captures are static so we can't use one here otherwise the last page to
include the file is what wins due to the way include caching works
during generation.
@heironimus
Copy link

When I apply this PR, the docs still show "Deployment" for all the headers

@heironimus
Copy link

If I completely revert this commit it fixes this issue. I've added a comment there.

@velle
Copy link
Contributor

velle commented Feb 12, 2024

When I apply this PR, the docs still show "Deployment" for all the headers

Agree. If I manually remove {%- capture item_html -%} and {%- endcapture -%} from docs_contents.html, that does not fix it.

@mattr-
Copy link
Member Author

mattr- commented Feb 12, 2024

The fact that it's broken is why it's still a draft PR 馃槃

There's a better fix available now though so I'll close this one out.

@mattr- mattr- closed this Feb 12, 2024
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 this pull request may close these issues.

[Docs]: The Jekyll Docs site is broken and always displays "Deployment" as the page heading.
3 participants