Optimise performance state panel

home-assistant-js

@@ -1 +1 @@
-Subproject commit 28b2e7ab6e9fecef769382f462b3c5c7c9964c49
+Subproject commit 11a648c2931f4f381af439b1c52d9ff028bd0626

src/components/ha-cards.html

@@ -128,7 +128,11 @@
         type: Object,
       },
 
-      paneVisible: {
+      panelVisible: {
+        type: Boolean,
+      },
+
+      viewVisible: {
         type: Boolean,
       },
 
@@ -138,13 +142,19 @@
     },
 
     observers: [
-      'updateCards(columns, states, showIntroduction, paneVisible)',
+      'updateCards(columns, states, showIntroduction, panelVisible, viewVisible)',
     ],
 
-    updateCards: function (columns, states, showIntroduction, paneVisible) {
+    updateCards: function (columns, states, showIntroduction,
-      if (!paneVisible) {
+                           panelVisible, viewVisible) {
+      if (!panelVisible || !viewVisible) {
+        console.log('cancelling update cards',
+                    this.getAttribute('data-view') || 'default-view',
+                    panelVisible, viewVisible);
         return;
       }
+      console.log('updateCards', this.getAttribute('data-view') || 'default-view',
+                  panelVisible, viewVisible);
       this.debounce(
         'updateCards',
         function () { this.cards = this.computeCards(columns, states,
@@ -153,6 +163,11 @@
     },
 
     computeCards: function (columns, states, showIntroduction) {
+      // Things might have changed since it got scheduled.
+      if (!this.panelVisible || !this.viewVisible) {
+        return;
+      }
+      console.error('computeCards', this.getAttribute('data-view') || 'default-view', states.size, this.viewVisible)
       var hass = this.hass;
       var byDomain = states.groupBy(function (entity) { return entity.domain; });
       var hasGroup = {};

src/components/ha-view-tabs.html

@@ -1,93 +0,0 @@
-<link rel="import" href="../../bower_components/paper-tabs/paper-tabs.html">
-
-<link rel='import' href='../util/hass-behavior.html'>
-
-<dom-module id="ha-view-tabs">
-  <template>
-    <style>
-      :host {
-        display: block;
-        --paper-tabs-selection-bar-color: #FFF;
-      }
-      paper-tab {
-        text-transform: uppercase;
-      }
-    </style>
-
-    <paper-tabs
-      selected='[[currentView]]'
-      attr-for-selected='data-entity'
-      on-iron-select='viewSelected'
-      scrollable=''
-    >
-        <paper-tab data-entity='' on-tap='viewTapped'>[[locationName]]</paper-tab>
-        <template is='dom-repeat' items='[[views]]'>
-          <paper-tab data-entity$='[[item.entityId]]' on-tap='viewTapped'>
-            <template is='dom-if' if='[[item.attributes.icon]]'>
-              <iron-icon icon='[[item.attributes.icon]]'></iron-icon>
-            </template>
-            <template is='dom-if' if='[[!item.attributes.icon]]'>
-              [[item.entityDisplay]]
-            </template>
-          </paper-tab>
-        </template>
-    </paper-tabs>
-  </template>
-</dom-module>
-
-<script>
-Polymer({
-  is: 'ha-view-tabs',
-
-  behaviors: [window.hassBehavior],
-
-  properties: {
-    hass: {
-      type: Object,
-    },
-
-    locationName: {
-      type: String,
-      bindNuclear: function (hass) { return hass.configGetters.locationName; },
-    },
-
-    currentView: {
-      type: String,
-      bindNuclear: function (hass) {
-        return [
-          hass.viewGetters.currentView,
-          function (view) { return view || ''; },
-        ];
-      },
-    },
-
-    views: {
-      type: Array,
-      bindNuclear: function (hass) {
-        return [
-          hass.viewGetters.views,
-          function (views) {
-            return views.valueSeq()
-              .sortBy(function (view) { return view.attributes.order; })
-              .toArray();
-          },
-        ];
-      },
-    },
-  },
-
-  viewTapped: function () {
-    this.fire('view-tapped');
-  },
-
-  viewSelected: function (ev) {
-    var view = ev.detail.item.getAttribute('data-entity') || null;
-    var current = this.currentView || null;
-    if (view !== current) {
-      this.async(function () {
-        this.hass.viewActions.selectView(view);
-      }.bind(this), 0);
-    }
-  },
-});
-</script>

src/layouts/home-assistant-main.html

@@ -30,7 +30,7 @@
         attr-for-selected='id'
         fallback-selection='panel-resolver'
         selected='[[activePane]]'
-        selected-attribute='pane-visible'
+        selected-attribute='panel-visible'
       >
         <partial-cards
           id='states'

src/layouts/partial-cards.html

@@ -1,6 +1,7 @@
 <link rel="import" href="../../bower_components/polymer/polymer.html">
 <link rel="import" href="../../bower_components/iron-icon/iron-icon.html">
 <link rel="import" href="../../bower_components/paper-icon-button/paper-icon-button.html">
+<link rel="import" href="../../bower_components/paper-tabs/paper-tabs.html">
 
 <link rel="import" href="../../bower_components/app-layout/app-header-layout/app-header-layout.html">
 <link rel="import" href="../../bower_components/app-layout/app-scroll-effects/app-scroll-effects.html">
@@ -9,7 +10,6 @@
 
 <link rel="import" href="../components/ha-menu-button.html">
 <link rel="import" href="../components/ha-cards.html">
-<link rel="import" href="../components/ha-view-tabs.html">
 
 <dom-module id="partial-cards">
   <template>
@@ -24,9 +24,10 @@
         background-color: #E5E5E5;
       }
 
-      ha-view-tabs {
+      paper-tabs {
         margin-left: 12px;
         --paper-tabs-selection-bar-color: #FFF;
+        text-transform: uppercase;
       }
     </style>
 
@@ -34,7 +35,7 @@
       <app-header effects="waterfall" condenses fixed>
         <app-toolbar>
           <ha-menu-button narrow='[[narrow]]' show-menu='[[showMenu]]'></ha-menu-button>
-          <div main-title>[[computeTitle(hasViews, locationName)]]</div>
+          <div main-title>[[computeTitle(views, locationName)]]</div>
           <paper-icon-button
             icon="mdi:refresh"
             class$="[[computeRefreshButtonClass(isFetching)]]"
@@ -45,20 +46,60 @@
             on-tap="handleListenClick"></paper-icon-button>
         </app-toolbar>
 
-        <ha-view-tabs
+        <div sticky>
-          sticky
+          <paper-tabs
-          hass='[[hass]]'
+            scrollable
-          on-view-tapped='scrollToTop'
+            selected='[[currentView]]'
-        ></ha-view-tabs>
+            attr-for-selected='data-entity'
+            on-iron-select='handleViewSelected'
+          >
+            <paper-tab
+              data-entity=''
+              on-tap='handleViewTapped'
+            >[[locationName]]</paper-tab>
+            <template is='dom-repeat' items='[[views]]'>
+              <paper-tab
+                data-entity$='[[item.entityId]]'
+                on-tap='handleViewTapped'
+              >
+                <template is='dom-if' if='[[item.attributes.icon]]'>
+                  <iron-icon icon='[[item.attributes.icon]]'></iron-icon>
+                </template>
+                <template is='dom-if' if='[[!item.attributes.icon]]'>
+                  [[item.entityDisplay]]
+                </template>
+              </paper-tab>
+            </template>
+          </paper-tabs>
+        </div>
       </app-header>
 
+      <iron-pages
+        attr-for-selected='data-view'
+        selected='[[currentView]]'
+        selected-attribute='view-visible'
+      >
         <ha-cards
+          data-view=''
           show-introduction='[[computeShowIntroduction(currentView, introductionLoaded, states)]]'
           states='[[states]]'
-        columns='[[columns]]'
+          columns='[[_columns]]'
           hass='[[hass]]'
-        pane-visible='[[paneVisible]]'
+          panel-visible='[[panelVisible]]'
         ></ha-cards>
+
+        <template is='dom-repeat' items='[[views]]'>
+          <ha-cards
+            data-view$='[[item.entityId]]'
+            show-introduction='[[computeShowIntroduction(currentView, introductionLoaded, states)]]'
+            states='[[states]]'
+            columns='[[_columns]]'
+            hass='[[hass]]'
+            panel-visible='[[panelVisible]]'
+          ></ha-cards>
+        </template>
+
+      </iron-pages>
     </app-header-layout>
   </template>
 
@@ -80,9 +121,20 @@ Polymer({
       value: false,
     },
 
-    paneVisible: {
+    showMenu: {
       type: Boolean,
       value: false,
+      observer: 'handleWindowChange',
+    },
+
+    panelVisible: {
+      type: Boolean,
+      value: false,
+    },
+
+    _columns: {
+      type: Number,
+      value: 1,
     },
 
     isFetching: {
@@ -126,12 +178,6 @@ Polymer({
       },
     },
 
-    showMenu: {
-      type: Boolean,
-      value: false,
-      observer: 'windowChange',
-    },
-
     currentView: {
       type: String,
       bindNuclear: function (hass) {
@@ -142,12 +188,16 @@ Polymer({
       },
     },
 
-    hasViews: {
+    views: {
-      type: Boolean,
+      type: Array,
       bindNuclear: function (hass) {
         return [
           hass.viewGetters.views,
-          function (views) { return views.size > 0; },
+          function (views) {
+            return views.valueSeq()
+              .sortBy(function (view) { return view.attributes.order; })
+              .toArray();
+          },
         ];
       },
     },
@@ -158,37 +208,31 @@ Polymer({
         return hass.viewGetters.currentViewEntities;
       },
     },
-
-    columns: {
-      type: Number,
-      value: 1,
-    },
   },
 
   created: function () {
-    var sizes = [];
+    var sizes = [300, 600, 900, 1200];
     var col;
-    this.windowChange = this.windowChange.bind(this);
+    this.handleWindowChange = this.handleWindowChange.bind(this);
-    for (col = 0; col < 5; col++) {
-      sizes.push(300 + (col * 300));
-    }
     this.mqls = sizes.map(function (width) {
       var mql = window.matchMedia('(min-width: ' + width + 'px)');
-      mql.addListener(this.windowChange);
+      mql.addListener(this.handleWindowChange);
       return mql;
     }.bind(this));
   },
 
   detached: function () {
     this.mqls.forEach(function (mql) {
-      mql.removeListener(this.windowChange);
+      mql.removeListener(this.handleWindowChange);
     });
   },
 
-  windowChange: function () {
+  handleWindowChange: function () {
-    var matchColumns = this.mqls.reduce(function (cols, mql) { return cols + mql.matches; }, 0);
+    var matchColumns = this.mqls.reduce(function (cols, mql) {
+      return cols + mql.matches;
+    }, 0);
     // Do -1 column if the menu is docked and open
-    this.columns = Math.max(1, matchColumns - (!this.narrow && this.showMenu));
+    this._columns = Math.max(1, matchColumns - (!this.narrow && this.showMenu));
   },
 
   /**
@@ -238,24 +282,31 @@ Polymer({
     this.hass.voiceActions.listen();
   },
 
-  computeMenuButtonClass: function (narrow, showMenu) {
+  handleViewSelected: function (ev) {
-    return !narrow && showMenu ? 'menu-icon invisible' : 'menu-icon';
+    var view = ev.detail.item.getAttribute('data-entity') || null;
+    var current = this.currentView || null;
+    if (view !== current) {
+      this.async(function () {
+        console.warn('changing view')
+        this.hass.viewActions.selectView(view);
+      }.bind(this), 0);
+    }
+  },
+
+  handleViewTapped: function (ev) {
+    this.scrollToTop();
   },
 
   computeRefreshButtonClass: function (isFetching) {
     return isFetching ? 'ha-spin' : '';
   },
 
-  computeTitle: function (hasViews, locationName) {
+  computeTitle: function (views, locationName) {
-    return hasViews ? 'Home Assistant' : locationName;
+    return views.length > 0 ? 'Home Assistant' : locationName;
   },
 
   computeShowIntroduction: function (currentView, introductionLoaded, states) {
     return currentView === '' && (introductionLoaded || states.size === 0);
   },
-
-  computeHasViews: function (views) {
-    return views.length > 0;
-  },
 });
 </script>

src/layouts/partial-panel-resolver.html

@@ -84,7 +84,12 @@ Polymer({
   },
 
   panelChanged: function (panel) {
-    if (!panel) return;
+    if (!panel) {
+      if (this.$.panel.lastChild) {
+        this.$.panel.removeChild(this.$.panel.lastChild);
+      }
+      return;
+    }
 
     this.resolved = false;
     this.errorLoading = false;