Skip to content

Commit 5655661

Browse files
authored
fix(cdk/drag-drop): remove preview after animate to placeholder animation completes (#29439)
* fix(cdk/drag-drop): remove preview after animate to placeholder animation completes The comparison between the event target element and this._preview never returned true because this._preview is a class, and we need to compare its element instead. This regression was introduced in commit ebab924#diff-65b82ba5c290701d792881e9e389e8f1961848abb740bb20940d4383dfae4eacL121-R117. * fixup! fix(cdk/drag-drop): remove preview after animate to placeholder animation completes Fix linting error
1 parent 23461b9 commit 5655661

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

src/cdk/drag-drop/drag-ref.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1106,7 +1106,9 @@ export class DragRef<T = any> {
11061106
const handler = ((event: TransitionEvent) => {
11071107
if (
11081108
!event ||
1109-
(_getEventTarget(event) === this._preview && event.propertyName === 'transform')
1109+
(this._preview &&
1110+
_getEventTarget(event) === this._preview.element &&
1111+
event.propertyName === 'transform')
11101112
) {
11111113
this._preview?.removeEventListener('transitionend', handler);
11121114
resolve();

src/cdk/drag-drop/preview-ref.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,10 @@ export class PreviewRef {
3939
/** Reference to the preview element. */
4040
private _preview: HTMLElement;
4141

42+
get element(): HTMLElement {
43+
return this._preview;
44+
}
45+
4246
constructor(
4347
private _document: Document,
4448
private _rootElement: HTMLElement,

0 commit comments

Comments
 (0)