Conversation
|
Preview available at https://egui-pr-preview.github.io/pr/8018-emilkscroll-fade View snapshot changes at kitdiff |
emilk
force-pushed
the
emilk/scroll-fade
branch
2 times, most recently
from
9c58b24 to
cce52ab
Compare
| return; | ||
| } | ||
|
|
||
| let bg = ui.visuals().panel_fill; // a bit iffy |
There was a problem hiding this comment.
I think we need a better way to solve this if we enable it by default
There was a problem hiding this comment.
Maybe we could do this using UiStack? Frame could set a background color property there and we could read it in the scroll area? That feels fairly simple and relatively low overhead
There was a problem hiding this comment.
Nice idea! I'll investigate that
There was a problem hiding this comment.
We already have UiStackInfo::frame, which is set by Frame, so I think we have all we need
There was a problem hiding this comment.
emilk
force-pushed
the
emilk/scroll-fade
branch
from
f2bdb70 to
7dff943
Compare
## Before: <img width="381" height="307" alt="image" src="https://github.com/user-attachments/assets/0528ae2a-44bf-4d9e-89a4-c3f4ab438eb2" /> It is very hard here to realize this is a scrollable area ## After <img width="383" height="310" alt="image" src="https://github.com/user-attachments/assets/9e0ee6de-8b96-4e5c-a505-f57977010990" /> The fade at the bottom tells the user they should try scrolling. You can turn if off with `style.spacing.scroll.fade.enabled`
* Follow-up to emilk#8018
2 participants
Before:
It is very hard here to realize this is a scrollable area
After
The fade at the bottom tells the user they should try scrolling.
You can control it with
style.spacing.scroll.fade