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

feat(cdkDrag): cdkDragExclude to mark a part of the tree as should not be #29827

Open
deefdragon opened this issue Oct 4, 2024 · 0 comments
Open
Labels
area: cdk/drag-drop feature This issue represents a new feature or feature request rather than a bug or bug fix P4 A relatively minor issue that is not relevant to core functions

Comments

@deefdragon
Copy link

Feature Description

Create a Directive (tentatively named cdkDragExclude) that when applied to an element does the effective inverse of cdkDragHandle. That element and it's children will not be able to be dragged/not cause the cdkDrag'ed item to be dragged.

As an alternative to a new directive, It should be possible to accomplish the same goal without adding a new directive by making it so that a cdkDragHandle that's disabled, inside another cdkDragHandle, (or a cdkDrag that's disabled inside another cdkDrag), will result in that element being un-dragable. As it stands, a cdkDrag in a cdkDrag will result in the parent one being moved, and a cdkDragHandle in a cdkDragHandle has no impact. I'm not sure how changing the interactions like that needs to be considered from an API stability point, and I'm also unsure if that's an acceptable option as it is not very intuitive.

Use Case

This is specifically targeted at parts of a component/component tree that have interactive elements of their own. (or where one might wish there to be interactivity)

  • Text inputs: Unable to highlight the text on an input with a mouse.
  • Text Area: Unable to change the size of a text area input as dragging the anchor in the bottom right corner results in the node moving instead
  • Sliders: Sliders become difficult/nearly impossible to use
  • Simply Highlighting text: Highlighting text in an element becomes difficult as the node itself will move instead of allowing the text to be selected.

This was discussed previously in #12929 and there was a response of "You can use the cdkDragHandle directive to configure which element should be draggable". The complexity of certain draggable components makes this nearly untenable under some circumstances unfortunately.

If I need to disable dragging on a single sibling element, I have to add cdkDragHandle to all other siblings of that element, all siblings of the element's parent, the parents parent, and so on until I reach the root dragable element itself. That process alone does not account for any margin on the element, which would have to be replaced with margin elements that also have cdkDragHandle, or other such hacks.

This only gets more complicated and unweildly the more elements you have that require disabling of the dragging.

@deefdragon deefdragon added feature This issue represents a new feature or feature request rather than a bug or bug fix needs triage This issue needs to be triaged by the team labels Oct 4, 2024
@crisbeto crisbeto added P4 A relatively minor issue that is not relevant to core functions area: cdk/drag-drop and removed needs triage This issue needs to be triaged by the team labels Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: cdk/drag-drop feature This issue represents a new feature or feature request rather than a bug or bug fix P4 A relatively minor issue that is not relevant to core functions
Projects
None yet
Development

No branches or pull requests

2 participants