Skip to content

fix(docs): add display block to sidebar category elements#53

Merged
jonathanperis merged 1 commit into
mainfrom
fix/docs-sidebar-vertical-layout
May 3, 2026
Merged

fix(docs): add display block to sidebar category elements#53
jonathanperis merged 1 commit into
mainfrom
fix/docs-sidebar-vertical-layout

Conversation

@jonathanperis
Copy link
Copy Markdown
Owner

@jonathanperis jonathanperis commented May 3, 2026

Summary

  • Add display: block to .sidebar-category-group, ul, and li
  • Ensures vertical stacking of nav items in categorized sidebar
  • Fixes horizontal wrapping issue on deployed site

Changes

  • docs/src/styles/docs.css: +5 lines

Verification

  • npm run build

Summary by CodeRabbit

  • Style
    • Improved documentation sidebar styling with explicit display properties for sidebar components and category groups, enhancing layout consistency and visual rendering reliability.

Add explicit display:block to .sidebar-category-group, ul, and li
to ensure vertical stacking of nav items in categorized sidebar
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 3, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: f0a594ad-7553-4082-9820-1f4e43df2bda

📥 Commits

Reviewing files that changed from the base of the PR and between 9761b43 and dd1aa3d.

📒 Files selected for processing (1)
  • docs/src/styles/docs.css

📝 Walkthrough

Walkthrough

CSS styling for documentation sidebar category groups was updated to explicitly set display: block on the category group container, nested unordered lists, and list items while preserving existing margin properties.

Changes

Sidebar Category Group Styling

Layer / File(s) Summary
Display Styling
docs/src/styles/docs.css
Added explicit display: block properties to .sidebar-category-group, its nested ul, and li children; preserved margin-bottom and margin-top properties.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title clearly and accurately summarizes the main change: adding display block CSS properties to sidebar category elements to fix layout issues.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/docs-sidebar-vertical-layout

Review rate limit: 1/10 review remaining, refill in 52 minutes and 41 seconds.

Comment @coderabbitai help to get the list of available commands and usage tips.

@jonathanperis jonathanperis merged commit 28b0321 into main May 3, 2026
5 checks passed
@jonathanperis jonathanperis deleted the fix/docs-sidebar-vertical-layout branch May 3, 2026 02:59
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.

1 participant