-
Notifications
You must be signed in to change notification settings - Fork 6.8k
feat(drag-drop): add the ability to disable dragging #13722
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
Conversation
Just curious: Why is it, that this feature is targeted for the next minor release while other new features are targeted for the next patch? Is this more complex in some way? |
We have a rule that it’s mostly bug fixes that go into patch releases, whereas features go into minor. |
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
hey, when will this pr be merged.. |
@crisbeto passes presubmit, just needs rebase |
Hi @crisbeto! This PR has merge conflicts due to recent upstream merges. |
858af25
to
d5a885b
Compare
Rebased. |
d5a885b
to
bd56a49
Compare
Adds inputs to `cdkDrag`, `cdkDropList` and `cdkDragHandle` that allows for dragging to be disabled through those specific elements. Fixes angular#13651.
any update? |
Adds inputs to `cdkDrag`, `cdkDropList` and `cdkDragHandle` that allows for dragging to be disabled through those specific elements. Fixes #13651.
Drag can be disabled using cdkDragDisabled property. How do i disable drop ? I wish there were a cdkDropDisabled property too. Though cdkDragDisabled is introduced in 7.1.0. expecting more option soon. |
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. |
Adds inputs to
cdkDrag
,cdkDropList
andcdkDragHandle
that allows for dragging to be disabled through those specific elements.Fixes #13651.