From fb49e9ec35ee963cf9204aadac4a1e4aa919661e Mon Sep 17 00:00:00 2001
From: Matt Mo <matt@montoyatech.com>
Date: Wed, 6 Jan 2021 08:51:24 -0800
Subject: [PATCH] Removed the on disconnected callback for elements since the
 only subscriber is the template and the template is already deconstructed by
 the element so there is no need to call it twice via the callback.

---
 ignite-element.js  | 7 -------
 ignite-template.js | 6 ------
 2 files changed, 13 deletions(-)

diff --git a/ignite-element.js b/ignite-element.js
index 9d70c10..c4bbd5f 100644
--- a/ignite-element.js
+++ b/ignite-element.js
@@ -51,7 +51,6 @@ class IgniteElement extends HTMLElement {
         super();
 
         this.elementConnected = false;
-        this.onDisconnected = null;
         this.template = null;
         this.elements = [];
         this.createProperties();
@@ -247,12 +246,6 @@ class IgniteElement extends HTMLElement {
                 this.template.deconstruct();
             }
 
-            //If we have a onDisconnected callback, call it and then remove the reference.
-            if (this.onDisconnected) {
-                this.onDisconnected();
-                this.onDisconnected = null;
-            }
-
             //Detach the after render callback
             this.readyCallback.disconnect();
 
diff --git a/ignite-template.js b/ignite-template.js
index abffa1e..2470e2e 100644
--- a/ignite-template.js
+++ b/ignite-template.js
@@ -667,12 +667,6 @@ class IgniteTemplate {
                 this.element.template = this;
             }
 
-            //If the element has a onDisconnected function, attach to it
-            //(This way if a custom element is removed we can deconstruct and cleanup)
-            if (this.element.onDisconnected !== undefined) {
-                this.element.onDisconnected = () => this.deconstruct();
-            }
-
             //Invoke any refs we have and pass back the element reference.
             this._refs.forEach((ref) => ref(this.element));
         }