Skip to content
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

AutoScroll overflow:hidden demo not working #1734

Open
ligne13 opened this issue Feb 1, 2020 · 6 comments
Open

AutoScroll overflow:hidden demo not working #1734

ligne13 opened this issue Feb 1, 2020 · 6 comments

Comments

@ligne13
Copy link

ligne13 commented Feb 1, 2020

Problem:

The given demo does not work (Chrome 79, Mac OS Catalina): https://jsbin.com/xecihez/edit?html,js
The list is not scrolling during a drag.

JSBin/JSFiddle demonstrating the problem:

https://jsbin.com/xecihez/edit?html,js,output

@waynevanson
Copy link
Contributor

This looks like a bug. Even with overflow off, it still doesn't scroll when I believe it should.

We'll have to look into this further.

@croossin
Copy link

@waynevanson Any updates on this?

@ahamilton9
Copy link

ahamilton9 commented Jul 29, 2020

I've only narrowed it down to a change made in the 1.8.0 release. If you change the JS Bin example to use /1.7.0/ instead of /master/ in the script tag, autoscroll works again. /1.8.0/ and later break. Will try to look soon, but it looks like that was a large release.

Possibly related to #1373?

Narrowed further to a change in commit 18cccba / PR #1386.

I've looked further into this, but I'm not close enough to this code to know what's going on anymore. Something with how the code since bubbleScroll was added handles finding and item's scroll-parent seems wonky, but I'm at a loss.

@simonsonga
Copy link

is this related to #1403 ?

@el-Rey-magico
Copy link

any updates on this?

1 similar comment
@gregveres
Copy link

any updates on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants