From ea57e71c8bb53528eb6b00d63099740a7554e629 Mon Sep 17 00:00:00 2001
From: Andrey <andrey-git@users.noreply.github.com>
Date: Wed, 21 Feb 2018 10:00:52 +0200
Subject: [PATCH] Optionally use Intersection Observer to late-load cards
 (#906)

---
 src/cards/ha-card-chooser.html | 43 +++++++++++++++++++++++++++++++---
 src/components/ha-cards.html   | 11 ++++-----
 2 files changed, 45 insertions(+), 9 deletions(-)

diff --git a/src/cards/ha-card-chooser.html b/src/cards/ha-card-chooser.html
index 84ee7558cf..163877a3b0 100644
--- a/src/cards/ha-card-chooser.html
+++ b/src/cards/ha-card-chooser.html
@@ -20,14 +20,51 @@ class HaCardChooser extends Polymer.Element {
     };
   }
 
-  cardDataChanged(newData) {
-    if (!newData) return;
-
+  _updateCard(newData) {
     window.hassUtil.dynamicContentUpdater(
       this, 'HA-' + newData.cardType.toUpperCase() + '-CARD',
       newData
     );
   }
+
+  cardDataChanged(newData) {
+    if (!newData) return;
+    // ha-entities-card is exempt from observer as it doesn't load heavy resources.
+    // and usually doesn't load external resources (except for entity_picture).
+    const eligibleToObserver =
+      (window.IntersectionObserver && newData.cardType !== 'entities');
+    if (!eligibleToObserver) {
+      if (this.observer) {
+        this.observer.unobserve(this);
+        this.observer = null;
+      }
+      this._updateCard(newData);
+      return;
+    }
+    if (!this.observer) {
+      this.observer = new IntersectionObserver((entries) => {
+        if (!entries.length) return;
+        if (entries[0].isIntersecting) {
+          this.style.height = '';
+          if (this._detachedChild) {
+            this.appendChild(this._detachedChild);
+            this._detachedChild = null;
+          }
+          this._updateCard(this.cardData); // Don't use 'newData' as it might have chnaged.
+        } else {
+          // Set the card to be 48px high. Otherwise if the card is kept as 0px height then all
+          // following cards would trigger the observer at once.
+          const offsetHeight = this.offsetHeight;
+          this.style.height = `${offsetHeight || 48}px`;
+          if (this.lastChild) {
+            this._detachedChild = this.lastChild;
+            this.removeChild(this.lastChild);
+          }
+        }
+      });
+      this.observer.observe(this);
+    }
+  }
 }
 customElements.define(HaCardChooser.is, HaCardChooser);
 </script>
diff --git a/src/components/ha-cards.html b/src/components/ha-cards.html
index a86732387b..61d1acbf1a 100644
--- a/src/components/ha-cards.html
+++ b/src/components/ha-cards.html
@@ -29,7 +29,8 @@
         overflow-x: hidden;
       }
 
-      .zone-card {
+      ha-card-chooser {
+        display: block;
         margin-left: 8px;
         margin-bottom: 8px;
       }
@@ -39,7 +40,7 @@
           padding-right: 0;
         }
 
-        .zone-card {
+        ha-card-chooser {
           margin-left: 0;
         }
       }
@@ -66,10 +67,8 @@
         <template is='dom-repeat' items='[[cards.columns]]' as='column'>
           <div class='column flex-1'>
             <template is='dom-repeat' items='[[column]]' as='card'>
-              <div class='zone-card'>
-                <ha-card-chooser card-data='[[card]]' hass='[[hass]]'
-                ></ha-card-chooser>
-              </div>
+              <ha-card-chooser card-data='[[card]]' hass='[[hass]]'
+              ></ha-card-chooser>
             </template>
           </div>
         </template>