From a7f9a5c4add9515ee0cc2e21a05414f78d5f0dd5 Mon Sep 17 00:00:00 2001
From: Paulus Schoutsen <paulus@paulusschoutsen.nl>
Date: Wed, 27 May 2015 22:02:58 -0700
Subject: [PATCH] Polymer 1.0: More info dialog bug fixes

---
 .../polymer/dialogs/more-info-dialog.html     | 42 ++++++++-----------
 1 file changed, 17 insertions(+), 25 deletions(-)

diff --git a/homeassistant/components/frontend/www_static/polymer/dialogs/more-info-dialog.html b/homeassistant/components/frontend/www_static/polymer/dialogs/more-info-dialog.html
index 88116365555..1eee563089e 100644
--- a/homeassistant/components/frontend/www_static/polymer/dialogs/more-info-dialog.html
+++ b/homeassistant/components/frontend/www_static/polymer/dialogs/more-info-dialog.html
@@ -2,9 +2,9 @@
 
 <link rel="import" href="../bower_components/paper-dialog/paper-dialog.html">
 <link rel="import" href="../bower_components/paper-dialog-scrollable/paper-dialog-scrollable.html">
-
-<!-- <link rel="import" href="ha-dialog.html"> -->
-
+<!-- <link rel="import" href="../bower_components/neon-animation/animations/slide-up-animation.html">
+<link rel="import" href="../bower_components/neon-animation/animations/slide-down-animation.html">
+ -->
 <link rel="import" href="../cards/state-card-content.html">
 <link rel="import" href="../components/state-history-charts.html">
 <link rel="import" href="../more-infos/more-info-content.html">
@@ -16,9 +16,10 @@
     }
   </style>
   <template>
-    <paper-dialog id="dialog" on-core-overlay-open="dialogOpenChanged" with-backdrop>
+    <!-- entry-animation='slide-up-animation' exit-animation='slide-down-animation' -->
+    <paper-dialog id="dialog" with-backdrop>
+      <h2><state-card-content state-obj="[[stateObj]]"></state-card-content></h2>
       <div>
-        <state-card-content state-obj="[[stateObj]]"></state-card-content>
         <template is='dom-if' if="[[hasHistoryComponent]]">
           <state-history-charts state-history="[[stateHistory]]"
             is-loading-data="[[isLoadingHistoryData]]"></state-history-charts>
@@ -50,7 +51,6 @@
 
       stateObj: {
         type: Object,
-        observer: 'reposition',
       },
 
       stateHistory: {
@@ -73,6 +73,11 @@
       },
     },
 
+    listeners: {
+      'iron-overlay-opened': 'onIronOverlayOpened',
+      'iron-overlay-closed': 'onIronOverlayClosed'
+    },
+
     componentStoreChanged: function(componentStore) {
       this.hasHistoryComponent = componentStore.isLoaded('history');
     },
@@ -101,12 +106,12 @@
       }
     },
 
-    dialogOpenChanged: function(ev) {
-      // we get CustomEvent, undefined and true/false from polymer…
-      console.log('debug', ev);
-      if (typeof ev === 'object') {
-        this.dialogOpen = ev.detail;
-      }
+    onIronOverlayOpened: function() {
+      this.dialogOpen = true;
+    },
+
+    onIronOverlayClosed: function() {
+      this.dialogOpen = false;
     },
 
     changeEntityId: function(entityId) {
@@ -121,19 +126,6 @@
       }
     },
 
-    /**
-     * Whenever the attributes change, the more info component can
-     * hide or show elements. We will reposition the dialog.
-     */
-    reposition: function(newVal, oldVal) {
-      // Only resize if already open
-      if(this.dialogOpen) {
-        this.debounce('resizeAfterLayoutChange', function() {
-          this.$.dialog.resizeHandler();
-        }.bind(this), 1000);
-      }
-    },
-
     show: function(entityId) {
       this.changeEntityId(entityId);