-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
fix(drag-drop): allow handle to be set on drag element if there is a different drag root #19559
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@crisbeto passes presubmit, just needs rebase |
5b7d03e
to
f9c3fb3
Compare
Rebased. |
@crisbeto after rebase it looks like integration_tests have a failure |
f9c3fb3
to
7b5ca5f
Compare
…different drag root Currently we only allow the drag handle to be a descendant of the drag element, but this can be inconvenient if the drag root is moved further up the DOM using `cdkDragRootElement`. These changes allow for a handle to be set on the drag element itself, if it has an alternate drag root. Fixes angular#19555.
7b5ca5f
to
bfa8941
Compare
Fixed the size failure. |
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. |
Currently we only allow the drag handle to be a descendant of the drag element, but this can be inconvenient if the drag root is moved further up the DOM using
cdkDragRootElement
. These changes allow for a handle to be set on the drag element itself, if it has an alternate drag root.Fixes #19555.