feat: add overflow-indicator-top and oveflow-indicator-bottom variants
#10182
Conversation
There was a problem hiding this comment.
I’m not sure why the previous visual test references screenshots didn't show the scrollbars. The one where both indicators are shown still doesn't show the scrollbars 🤷♂️
There was a problem hiding this comment.
We have hide-scrollbars.js file but IMO it can be removed. I'll handle that.
There was a problem hiding this comment.
UPD: removed that file from base styles but preserved for Lumo since it is tested with older Chrome version where scrollbars make the appearance of the indicators less obvious (I believe that’s the reason why they were hidden).
web-padawan
force-pushed
the
feat/scroller-overflow-indicators
branch
from
ee4ef2f to
b2b2461
Compare
|
|
This ticket/PR has been released with Vaadin 25.0.0-alpha12. |
Allow developers to choose if they want to show the indicator at the top, bottom, or both edges of Scroller and Virtual List.