-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
cdkDropList has wrong container after scrolling #17144
Labels
P3
An issue that is relevant to core functions, but does not impede progress. Important, but not urgent
Comments
crisbeto
added a commit
to crisbeto/material2
that referenced
this issue
Sep 21, 2019
… into view via the parent In angular#16681 we added some logic that updates the cached dimensions if the document is scrolled while dragging, however it didn't account for any scrollable parent nodes that might've been scrolled. These changes switch the event listener to use capturing so that we can pick up all of the elements being scrolled. Fixes angular#17144.
andrewseguin
pushed a commit
that referenced
this issue
Sep 27, 2019
… into view via the parent (#17162) In #16681 we added some logic that updates the cached dimensions if the document is scrolled while dragging, however it didn't account for any scrollable parent nodes that might've been scrolled. These changes switch the event listener to use capturing so that we can pick up all of the elements being scrolled. Fixes #17144.
andrewseguin
pushed a commit
that referenced
this issue
Sep 30, 2019
… into view via the parent (#17162) In #16681 we added some logic that updates the cached dimensions if the document is scrolled while dragging, however it didn't account for any scrollable parent nodes that might've been scrolled. These changes switch the event listener to use capturing so that we can pick up all of the elements being scrolled. Fixes #17144. (cherry picked from commit 2c1c30a)
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
P3
An issue that is relevant to core functions, but does not impede progress. Important, but not urgent
When there are multiple cdkDropLists on a page and an item from the droplist is dropped on a target container that is only visible after scrolling it into view. The container that is referenced as the target container is not the correct container.
Here is an example in on stackblitz.
https://stackblitz.com/edit/angular-xoxgw2
Steps to reproduce:
Drag an item from a visible container.
Scroll to a container that was not visible before.
drop the item on the container
Expected Behavior
The target should be dropped on the correct container
Environment
Angular: 8.2.7
CDK/Material: 8.2.0
I already mentioned this before in issue #16648 . But after upgrading to version 8.2.0 with a fix #16681 the problem still occurs as shown in the example on stackblitz
The text was updated successfully, but these errors were encountered: