Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

Fix overview layout description to clarify TOC behavior

Summary

Corrects the documentation for the overview layout option in the page-level settings (frontmatter). The previous documentation incorrectly stated that overview layout was "without a table of contents", but the actual implementation shows the TOC by default.

Changes:

  • Updated overview layout description to clarify it shows TOC by default (50% wider than guide layout)
  • Added guidance to use hide-toc: true to hide the TOC if needed
  • Clarified that reference layout "always hides" the TOC (was "removes")
  • Fixed "full-width" description to "50% wider than guide" to match the API definition

Root cause: The public documentation contradicted both the implementation code (OverviewLayout.tsx renders {toc}) and the API definition (frontmatter.yml states "the table of contents is still visible").

Review & Testing Checklist for Human

  • Create a test page with layout: overview and verify the TOC is actually visible (I verified this in code but didn't test live)
  • Test that layout: overview combined with hide-toc: true correctly hides the TOC
  • Review the preview URL to ensure the updated documentation reads clearly and accurately
  • Confirm the "50% wider than guide" description is accurate (based on API definition, not visual testing)

Notes

The overview layout documentation incorrectly stated it was 'without a table of contents'.
This updates the documentation to accurately reflect that:
- Overview layout shows the TOC by default (50% wider than guide layout)
- Users can use hide-toc: true to hide it if needed
- Reference layout always hides the TOC
- This matches the actual implementation and API definition

Co-Authored-By: Devin Logan <[email protected]>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025

@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025

@jon-fern jon-fern self-requested a review November 7, 2025 20:08
@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025

@devalog devalog merged commit d1785f9 into main Nov 7, 2025
2 checks passed
@devalog devalog deleted the devin/1762545104-fix-overview-layout-docs branch November 7, 2025 20:31
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.

3 participants