fix(docs): align sidebar content vertically#1092
Conversation
Signed-off-by: PARTH-TUSSLE <parthgartan26feb@gmail.com>
There was a problem hiding this comment.
Code Review
This pull request updates the top padding of a sticky container in assets/scss/_styles_project.scss from 1rem to 2.15rem. The reviewer raised valid concerns regarding the use of a hardcoded magic number, which is fragile and could cause misalignment. Additionally, the reviewer pointed out potential layout issues on mobile devices where other sidebars reset their padding, and suggested using a shared SCSS variable or CSS custom property along with appropriate mobile overrides.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|
🚀 Preview deployment: https://docs.layer5.io/pr-preview/pr-1092/
|
|
@PARTH-TUSSLE Thank you for your contribution! Let's discuss this during the website call tomorrow at 5:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂 |
There was a problem hiding this comment.
I see that you have increased the padding of the right sidebar. Wouldn't that create more ugly whitespace though? What do you think about reducing the spacing on the left side instead?
There was a problem hiding this comment.
You're right, on it 🫡
There was a problem hiding this comment.
@Bharath314 Removed the unnecessary whitespace and improved responsiveness by using shared variable
Signed-off-by: PARTH-TUSSLE <parthgartan26feb@gmail.com>
Signed-off-by: PARTH-TUSSLE <parthgartan26feb@gmail.com>
|
Hey @Bharath314 , I have now completely refactored the styling system of the sidebars, created shared central variables that are being used by both sidebars for consistency, they are perfectly aligned and no unnecessary whitespace, would appreciate any follow up. |
KatalKavya96
left a comment
There was a problem hiding this comment.
Thanks @PARTH-TUSSLE , I reviewed the latest changes and checked the preview deployment.
- The desktop alignment looks good now, and the shared sidebar variables are a cleaner approach compared to the earlier hardcoded spacing fix. The unnecessary whitespace also seems to be removed.
However, I noticed a regression in mobile view: the content readability breaks because the page overflows and the full content cannot be viewed comfortably within a single phone viewport.
Could you please recheck the mobile responsive behaviour. The desktop view looks good so the remaining concern is mainly the mobile overflow/readability issue.
Thank you for the follow up, On it 🫡. |
Thanks for the update, @PARTH-TUSSLE. Just ping me once the further changes are pushed, and I’ll take another look at the preview. |
Signed-off-by: PARTH-TUSSLE <parthgartan26feb@gmail.com>
Signed-off-by: PARTH-TUSSLE <parthgartan26feb@gmail.com>
Hey @KatalKavya96 I've been working on this issue since morning, fixed the responsiveness and the overflow problem. And also , since the nav options weren't completely visible and were not fitting in the viewport, I made it scrollable. ->
Please look into it , would appreciate any follow up . |





Notes for Reviewers
Adjusted the sidebar spacing to align the content in the left and right sidebars more consistently.
This resolves the visual asymmetry where the left sidebar content started lower than the right sidebar content.
Preview of the fix :-

This PR fixes #1080
Signed commits