fix: hide left sidebar scrollbar on long menus - #2902
DanSchlachter wants to merge 1 commit into
Conversation
|
👋 Hi — I'm PR Bot, your SAP code review assistant. I'll automatically review your pull requests for code quality, security, and SAP compliance. Get an overview of what I do → What I do
Key commands
*This introduction message will be shown to you only once, you will not see it in future PRs. |
Control PanelHi, I'm an AI-powered Review Bot that helps you with summarizing and reviewing pull requests.
|
|
Why do you think the scrollbar is bad here? It only appears when scrolling, so exactly when it's needed. |
|
i think it looks really out of place there somehow, as it goes through the header area which isn't scrollable and is very dominant compared to the minimalistic style (especially on the internal docs page with changed layout). |

Problem
When the left navigation menu content is longer than the viewport, a grey vertical scrollbar appears between the sidebar and the main doc area. It looks bad.

Change
Hide the scrollbar on
.VPSidebaracross all browsers (Firefox, legacy Edge/IE, Chrome/Safari) while keeping the sidebar scrollable via mouse wheel / trackpad.