Iterate on scrollbars - Githubissues

JetBrains / jewel

An implementation of the IntelliJ look and feels in Compose for Desktop

Apache License 2.0

672 stars 34 forks source link

Iterate on scrollbars #445

Closed hamen closed 1 month ago

hamen commented 1 month ago

Based on the initial work in #419, we need to add:

[ ] On hover, scrollbar width grows to 14.dp
[x] Scrollbar track must be visible when the scrollbar is engaged
[ ] Track must be clickable to allow scrollToPosition or scrollByOnePage
[x] The scrollbar is hidden by default unless the user starts scrolling
[x] The scrollbar disappears after an inactivity period of N seconds

The following image shows how this works in the current non-Jewel implementation:

Screen Recording 2024-07-12 at 13 19 48

rock3r commented 1 month ago

Also needed is to check other OSes to make sure they look correct. They should look the same everywhere, according to the Int UI specs, but they didn't use to the last time I checked, and I'm not sure they do as of today.

Plus, macOS also allows customising whether they always show, always show when using a mouse, or only appear when scrolling. In every case, they must never overlap actual content in their non-active (the narrower, idle) state

rock3r commented 1 month ago

Swing reference code: com.intellij.ui.components.MacScrollBarUI and com.intellij.openapi.editor.impl.EditorImpl.MyScrollBar