Fix: Reduce sidebar width to improve content area alignment#791
Fix: Reduce sidebar width to improve content area alignment#791SouparnaChatterjee wants to merge 1 commit into
Conversation
|
Warning Review limit reached
More reviews will be available in 14 minutes and 36 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughAdds a new Possibly related PRs
Suggested labels
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
_sass/custom/custom.scss (1)
320-329: ⚡ Quick winConsider combining duplicate media queries for the same breakpoint.
A
@media (min-width: 50rem)block already exists at lines 246-273. Combining the sidebar/main rules into that existing block would improve maintainability and reduce duplication.♻️ Consolidate into existing media query
Move lines 322-328 into the existing
@media (min-width: 50rem)block at line 246, and remove the duplicate wrapper:At line 246, the block would become:
`@media` (min-width: 50rem) { + // Fix: reduce sidebar width and fix alignment (Issue `#709`) + .side-bar { + width: 11rem; + } + + .main { + margin-left: 11rem; + } + // Sidebar Navigation .site-nav { padding-left: 2rem;Then remove lines 320-329 from their current location.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 315e5d23-61d3-4c20-b1ff-425ff6843f2c
📒 Files selected for processing (2)
_includes/components/nav.html_sass/custom/custom.scss
|
@Nihal4777 LGTM |
f4ee180 to
b83d571
Compare
c92c04e to
906ec8f
Compare
906ec8f to
8022c05
Compare
Fixes #709
Changes done:
.side-barwidth from15.5rem/16.5remto11remin_sass/custom/custom.scssfor both medium (50rem) and large (66.5rem) breakpoints..mainmargin-leftvalues to match the new sidebar width at both breakpoints, preventing content overlap.Screenshots
Before: Sidebar consumed excessive horizontal space, leaving less room for main content.

After: Sidebar is more compact, main content area is wider and better utilizes screen space.

✅️ By submitting this PR, I have verified the following
Summary by CodeRabbit
Release Notes