-
Notifications
You must be signed in to change notification settings - Fork 77
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
padding on the container and renderFromLast #37
Comments
Hmm, that's a tricky one. Users can technically put any number of things before or after the One possible solution would be to set the initial |
Quick update, @runspired any thoughts here? If |
I need more context on this issue, as I understand it, is the layout this?
|
my instance is more like so
Then to keep my collection items from being flush I added padding to the scroll container. So let's say my padding was 20px all around, the |
@evoactivity can you use margin instead for this? The trouble with padding is it's relatively hard to distinguish between height and padding, but it's not too hard to distinguish between margin and (height + padding). |
@evoactivity this should be fixed for static heights on master, and for dynamic heights when #84 lands. |
@evoactivity this should be fixed now, closing unless you encounter additional issue :) |
Nice one @runspired, I'll let you know if I run into anything? |
So my scroll container has padding on it, and I want to load scrolled to the bottom so I've set
renderFromLast
to true, this seems to work except the scroll position doesn't seem to take into account the container top and bottom padding.I can work around this in my app with different css but I wonder if this should be handled by the library?
The text was updated successfully, but these errors were encountered: