fix: restore Docs.tsx from ui-optimization-suggestions branch#59
Conversation
Restore the complete Docs page component to match the ui-optimization-suggestions branch, including improved mobile sidebar, navigation enhancements, and UI optimizations. https://claude.ai/code/session_012KsDR5wEapUHh2cNNwryLm
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughWalkthroughThis change comprehensively refactors the Docs page layout, introducing mobile-aware sidebar positioning with backdrop overlay, restructuring the main content area, redesigning the right TOC sidebar with GitBook-style formatting, updating breadcrumb navigation, enhancing code block UI, improving typography across sections, and refining the search modal interactions. Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Poem
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. Comment |
📊 Performance Report
View detailed Lighthouse report in artifacts. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 482b4321c5
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| {/* Sidebar Navigation Content - Will add navigation items here */} | ||
| <div style={{ padding: '0 20px' }}> | ||
| <div style={{ | ||
| fontSize: '11px', | ||
| fontWeight: 600, | ||
| textTransform: 'uppercase', | ||
| letterSpacing: '0.5px', | ||
| color: colors.textMuted, | ||
| marginBottom: '12px', | ||
| }}> | ||
| Documentation | ||
| </div> | ||
| {/* Navigation items will be rendered here */} | ||
| </div> |
There was a problem hiding this comment.
Restore sidebar tree rendering in Docs navigation
The new sidebar layout no longer renders sidebarTree/TreeItem entries and only shows a static "Documentation" heading, so users cannot browse or switch between most docs pages from the left navigation (especially on desktop where this is the primary navigation surface). This is a functional regression from the previous implementation where all docs nodes were clickable in this area.
Useful? React with 👍 / 👎.
Restore the complete Docs page component to match the ui-optimization-suggestions branch, including improved mobile sidebar, navigation enhancements, and UI optimizations.
https://claude.ai/code/session_012KsDR5wEapUHh2cNNwryLm
Pull Request
Description
Type of Change
Related Issues
Changes Made
Screenshots/Videos
Before
After
Testing
Design System Compliance
#7C85EDappropriately (sparingly)Checklist
Additional Notes
Summary by CodeRabbit
New Features
Style