Merge pull request #238 from Esri/sketch-updates

Sketch updates
Esri · May 21, 2023 · ff7607d · ff7607d
2 parents 95168ce + a7b4f1d
commit ff7607d
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/src/components/map-draw-tools/map-draw-tools.tsx b/src/components/map-draw-tools/map-draw-tools.tsx
@@ -257,6 +257,18 @@ export class MapDrawTools {
     this._init();
   }
 
+  /**
+   * StencilJS: Called every time the component is disconnected from the DOM
+   *
+   * @returns void
+   */
+  disconnectedCallback(): void {
+    // cancel any existing create operations
+    this._sketchWidget.cancel();
+    // clear any current temp sketch
+    this._clearSketch();
+  }
+
   /**
    * Renders the component.
    */

diff --git a/src/components/public-notification/public-notification.tsx b/src/components/public-notification/public-notification.tsx
@@ -761,7 +761,9 @@ export class PublicNotification {
           // REFINE will only ever be 1 ISelectionSet
           Object.keys(cur.refineInfos).forEach(k => {
             const refineIds: IRefineIds = cur.refineInfos[k];
-            this._updateIds(k, refineIds.layerView, refineIds.addIds, prev);
+            if (refineIds.addIds.length > 0) {
+              this._updateIds(k, refineIds.layerView, refineIds.addIds, prev);
+            }
           });
         }
       }