fix(drag-drop): support cdkDropListHasAnchor placeholder styling#2110
fix(drag-drop): support cdkDropListHasAnchor placeholder styling#2110chintankavathia wants to merge 1 commit into
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a new drag-and-drop copy example for the tree view component, including the necessary SCSS style updates to support the cdkDropListHasAnchor feature. Feedback on the changes highlights a missing [cdkDragData] input in the site configuration tree template that would lead to a runtime error, as well as a redundant min-block-size property definition in the tree view item styles.
Adjust drag placeholder styles so that drop lists using `cdkDropListHasAnchor` keep the original item visible as an anchor in the source list while still hiding the placeholder in plain drop lists. Also make the anchor clone always render like the original item.
chintankavathia
force-pushed
the
fix/drag-drop/styles/has-anchor
branch
from
a90f437 to
4cb483c
Compare
|
With the current implementation, a drag to another tree that doesn't accept a drop can cause unintended re-ordering/sorting behavior. It would be good to document that adding And, this approach removes the source item from the tree when the drag is initiated which is confusing for the user. 
|
2 participants
Adjust drag placeholder styles so that drop lists using
cdkDropListHasAnchorkeep the original item visible as an anchor in the source list while still hiding the placeholder in plain drop lists. Also make the anchor clone always render like the original item.Related to #2106
Documentation.
Examples.
Dashboards Demo.
Playwright report.
Coverage Reports: