Skip to content

Fix the docs contents menu#9529

Closed
mattr- wants to merge 1 commit intomasterfrom
fix-contents-bug
Closed

Fix the docs contents menu#9529
mattr- wants to merge 1 commit intomasterfrom
fix-contents-bug

Conversation

@mattr-
Copy link
Copy Markdown
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
Copy Markdown

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

@heironimus
Copy link
Copy Markdown

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

@velle
Copy link
Copy Markdown
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
Copy Markdown
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
@jekyll jekyll locked and limited conversation to collaborators Feb 11, 2025
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.

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

4 participants