fix: resolve documentation sidebar overlap - #2124
Ayushgupta2408 wants to merge 3 commits into
Conversation
|
@Ayushgupta2408 is attempting to deploy a commit to the recode Team on Vercel. A member of the Team first needs to authorize it. |
|
Thank you for submitting your pull request! 🙌 We'll review it asap. The estimated time for response is 5–8 hrs. In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, if it's a Video -LEGEND, which helps speed up the review. If you have questions, reach out to LinkedIn. You can see in our Leaderboard in 20 min from now!😊 Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv. We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰 🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨ 📚Your perks for contribution to this community 👇🏻
Thanks again for your contribution! 😊 |
github-actions
Bot
added
in-review
|
✅ Synchronized metadata from Issue #2113:
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Its still the same. |
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Moderate layout and accessibility issues remain unresolved.
Get a fresh assessment by requesting another Copilot review.
Review effort: Lite
Findings: 4
Open (4)
What changed in this PR
Fixes documentation sidebar overlap and content clipping, with related FAQ markup and cleanup.
Changes:
- Updates responsive sidebar and documentation layout CSS.
- Adjusts blog sizing and FAQ presentation.
- Removes extraneous whitespace in the dashboard sidebar.
| File | Summary |
|---|---|
src/css/custom.css |
Moderate findings (2 votes each): fixed sidebar positioning may cause tablet overlap; overflow-x: hidden can clip embedded documentation content. |
src/components/faqs/faqs.tsx |
Moderate findings (3 votes each): heading hierarchy is incorrect, and h3 elements create invalid button markup; the latter also occurs on line 229. |
src/components/dashboard/Sidebar/DashboardSidebar.tsx |
Whitespace-only cleanup. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| <p | ||
| className="mb-2.5 inline-flex items-center gap-2 text-xs font-semibold tracking-widest uppercase" | ||
| <div className="mb-5 text-center lg:mb-6"> | ||
| <h3 |
| aria-controls={`faq-panel-${originalIndex}`} | ||
| > | ||
| <span>{faq.question}</span> | ||
| <h3 className="text-lg font-semibold md:text-xl m-0">{faq.question}</h3> |
| .theme-doc-sidebar-container { | ||
| width: 100% !important; | ||
| position: fixed !important; | ||
| z-index: 9999 !important; | ||
| padding: 12px !important; |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
✅ Synchronized metadata from Issue #2113:
|
Description
Fixes the documentation sidebar layout issue where the sidebar overlaps or cuts off the documentation content when navigating to the Docs section.
Fix #2113
Testing
Tested the documentation section locally.
Verified the sidebar and documentation content display correctly without overlap or clipping.
Ran the production build successfully.