When user in the bottom of scrollable list, scroll up is not working

[SimchaShats]

Bug: When user in the bottom of scrollable list, and drag bottom item to the top, scroll top is not working.

Expected behaviour: List should scroll up when user move draggable item to the top of the list

Actual behaviour: List is not scrollable

Steps to reproduce: Basic example from library with updates: https://codesandbox.io/s/3457zo2kn1

What version of React are you using: 16
What version of react-beautiful-dnd are you running: latest
What browser are you using: Chrome
Operating system: OSX (mac)

Demo

[alexreardon]

Please provide the information requested in our ISSUE_TEMPLATE or we cannot action this

[SimchaShats]

@alexreardon Updated

[alexreardon]

I cannot seem to reproduce what you are seeing

https://codesandbox.io/s/o4yljr6999

[alexreardon]

Here is another example that exercises the same use case:

https://react-beautiful-dnd.netlify.com/?selectedKind=Super%20simple&selectedStory=vertical%20list%20with%20scroll&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel

[SimchaShats]

Ah ok thanks, but what wrong with my example? I took there basic example from the library page. I had same issue in my project...

[alexreardon]

I cannot reproduce it on my machine with your example: https://codesandbox.io/s/3457zo2kn1

...?

[alexreardon]

I cannot reproduce it in FireFox or Chrome

[alexreardon]

Using the example you provided...

[SimchaShats]

Ah may be because I'm using mac, I attached gif from codesandbox that did an hour ago. And your example that you attached now is working on my machine

[SimchaShats]

Ah found, it doesn't work if there is exististing horizontal scroll, if remove it - it works. Not sure if it's a bug or no...

[alexreardon]

Can you please provide some more details?

…

On Mon, 9 Jul 2018 at 5:34 pm, Simcha Shats ***@***.***> wrote: Ah found, it doesn't work if there is exististing horizontal scroll, if remove it - it works. Not sure if it's a bug or now... — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#628 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACFN7Y60sJ_pNkNm73Q_OFbjz30rzevgks5uEwd9gaJpZM4VHHh8> .

[SimchaShats]

Here the example: https://ufile.io/4qd3a

[alexreardon]

I found the issue! When you are using overflow: scroll it is adding a horizontal scroll bar. This difference is messing with the auto scrolling calculations.

Try using one of the following

• overflow: auto
• overflow-y: scroll
• overflow-y: auto

All of these will work how you expect them to!

Not sure if it is worth looking into the issue caused by the redundant scroll bar. The issue is caused because the redundant scroll bar is messing with our internal dimension model

[alexreardon]

I am keen to solve the underlying issue though :)

[SimchaShats]

ok thanks