From 8c2440765383f8ba9d6051ea260a39122b8fc2fd Mon Sep 17 00:00:00 2001 From: Liam DeBeasi Date: Thu, 16 Sep 2021 12:14:32 -0400 Subject: [PATCH] fix(item-sliding): item-sliding accounts for multiple ion-item elements --- .../components/item-sliding/item-sliding.tsx | 7 +++++ .../item-sliding/test/basic/index.html | 26 +++++++++++++++++++ 2 files changed, 33 insertions(+) diff --git a/core/src/components/item-sliding/item-sliding.tsx b/core/src/components/item-sliding/item-sliding.tsx index cd6b67841c3..b2a854975c2 100644 --- a/core/src/components/item-sliding/item-sliding.tsx +++ b/core/src/components/item-sliding/item-sliding.tsx @@ -270,6 +270,13 @@ export class ItemSliding implements ComponentInterface { } private onStart() { + /** + * We need to query for the ion-item + * every time the gesture starts. Developers + * may toggle ion-item elements via *ngIf. + */ + this.item = this.el.querySelector('ion-item'); + // Prevent scrolling during gesture this.disableContentScrollY(); diff --git a/core/src/components/item-sliding/test/basic/index.html b/core/src/components/item-sliding/test/basic/index.html index c5ebbbc1007..4a4d6f2784c 100644 --- a/core/src/components/item-sliding/test/basic/index.html +++ b/core/src/components/item-sliding/test/basic/index.html @@ -41,6 +41,7 @@ Open Item Start Open Item End Open Item with only one side + Swap dynamic item @@ -369,6 +370,17 @@

RIGHT/LEFT side - many buttons

+ + + Dynamic First Item + + + + First Item Options + + + +

Normal ion-item (no sliding)

@@ -387,6 +399,20 @@

Normal button (no sliding)