bek-BLUE-40: hide scroll bar #14
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR implements a feature to hide and show scrollbars in the layout sidebar based on user interaction. The scrollbar is hidden by default and appears on hover for devices with fine pointer input.
Key Changes:
- Added CSS styling to hide scrollbars by default and show them on hover
- Included a fade-out mask effect for the non-hover state
- Updated Visual Studio version metadata
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| BlueBlazor/Components/Layout/Layout.razor.css | Added media query-based scrollbar visibility styling with hover interactions |
| BlueBlazor.sln | Updated Visual Studio version from 18 to 17 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
lgk-bsw
left a comment
lgk-bsw
left a comment
There was a problem hiding this comment.
CSS shouldn't be in Blue Blazor, but in Blue Web. Correct place would be: https://github.com/bruegmann/blue-web/blob/main/dist/styles/_layout.scss
When scrolled to the end of the sidebar, the fade shouldn't be there anymore:
Maybe you could solve this using CSS scoll-state(). Check browser compatibility. If the current browser isn't supported, the whole effect shouldn't be there.
A smooth transition on hover for the fade would be nice. But only if user doesn't prefer reduced motion. You can check this with CSS Media Query: https://developer.mozilla.org/en-US/docs/Web/CSS/Reference/At-rules/@media/prefers-reduced-motion
|
Wurde jetzt durch bruegmann/blue-web#7 umgesetzt. |
No description provided.