BrightspaceUI / core

A collection of accessible, free, open-source web components for building Brightspace applications.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

d2l-scroll-wrapper does not work as expected with d2l-list

MitchelPaulin opened this issue · comments

See this branch for how its being used https://github.com/Brightspace/d2l-rubric/tree/US135547-Add-Back-Scroll-Wrapper

The vertical portion will always attempt to scroll even when there is no need, in the screenshot you can see that the group is the correct size with no hidden elements causing the vertical scroll to kick in, so not sure why it does. I found a simple fix, if I just set overflow-y: hidden in the scroll wrapper it behaves like it should, though not sure if adding an API option to set the vertical scrolling behavior of the scroll wrapper would be an acceptable solution.

Animation4
Untitled

This is fixed with the above merged change.