Skip to content

fix: include empty subsections in auto-generated sidebar#1885

Merged
markdumay merged 2 commits intomainfrom
develop
Apr 21, 2026
Merged

fix: include empty subsections in auto-generated sidebar#1885
markdumay merged 2 commits intomainfrom
develop

Conversation

@markdumay
Copy link
Copy Markdown
Collaborator

The live-pages partial only added subsection list pages by walking the .Parent of regular (leaf) pages. Subsections containing only _index.md (no regular descendants) were silently dropped from the hierarchical menu. Now, when nested is true and group-by is section, walk $sectionPage.Sections recursively so every descendant subsection is included, then deduplicate with the existing parent-based entries.

The live-pages partial only added subsection list pages by walking the
.Parent of regular (leaf) pages. Subsections containing only _index.md
(no regular descendants) were silently dropped from the hierarchical
menu. Now, when nested is true and group-by is section, walk
$sectionPage.Sections recursively so every descendant subsection is
included, then deduplicate with the existing parent-based entries.
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 21, 2026

Deploy Preview for gethinode-demo ready!

Name Link
🔨 Latest commit 28c0535
🔍 Latest deploy log https://app.netlify.com/projects/gethinode-demo/deploys/69e72cb20e3e3700084fecbe
😎 Deploy Preview https://deploy-preview-1885--gethinode-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@markdumay markdumay merged commit 07bc3c0 into main Apr 21, 2026
15 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.8.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant