Fix sun more info card

src/more-infos/more-info-sun.html

@@ -4,18 +4,14 @@
 
 <dom-module id="more-info-sun">
   <template>
-    <div class='data-entry layout justified horizontal' id='rising'>
+    <template is='dom-repeat' items='[[computeOrder(risingDate, settingDate)]]'>
-      <div class='key'>
+      <div class='data-entry layout justified horizontal' id='rising'>
-        Rising <relative-ha-datetime datetime-obj="[[risingDate]]"></relative-ha-datetime>
+        <div class='key'>
+          <span>[[itemCaption(item)]]</span>
+          <relative-ha-datetime datetime-obj="[[itemDate(item)]]"></relative-ha-datetime>
+        </div>
+        <div class='value'>[[itemValue(item)]]</div>
       </div>
-      <div class='value'>[[risingTime]]</div>
+    </template>
-    </div>
-
-    <div class='data-entry layout justified horizontal' id='setting'>
-      <div class='key'>
-        Setting <relative-ha-datetime datetime-obj="[[settingDate]]"></relative-ha-datetime>
-      </div>
-      <div class='value'>[[settingTime]]</div>
-    </div>
   </template>
 </dom-module>

src/more-infos/more-info-sun.js

@@ -10,39 +10,40 @@ export default new Polymer({
   properties: {
     stateObj: {
       type: Object,
-      observer: 'stateObjChanged',
     },
 
     risingDate: {
       type: Object,
+      computed: 'computeRising(stateObj)',
     },
 
     settingDate: {
       type: Object,
-    },
+      computed: 'computeSetting(stateObj)',
-
-    risingTime: {
-      type: String,
-    },
-
-    settingTime: {
-      type: String,
     },
   },
 
-  stateObjChanged() {
+  computeRising(stateObj) {
-    this.risingDate = parseDateTime(this.stateObj.attributes.next_rising);
+    return parseDateTime(stateObj.attributes.next_rising);
-    this.risingTime = formatTime(this.risingDate);
+  },
 
-    this.settingDate = parseDateTime(this.stateObj.attributes.next_setting);
+  computeSetting(stateObj) {
-    this.settingTime = formatTime(this.settingDate);
+    return parseDateTime(stateObj.attributes.next_setting);
+  },
 
-    const root = Polymer.dom(this);
+  computeOrder(risingDate, settingDate) {
+    return risingDate > settingDate ?  ['set', 'ris'] : ['ris', 'set'];
+  },
 
-    if (self.risingDate > self.settingDate) {
+  itemCaption(type) {
-      root.appendChild(this.$.rising);
+    return type === 'ris' ? 'Rising ' : 'Setting ';
-    } else {
+  },
-      root.appendChild(this.$.setting);
+
-    }
+  itemDate(type) {
+    return type === 'ris' ? this.risingDate : this.settingDate;
+  },
+
+  itemValue(type) {
+    return formatTime(this.itemDate(type));
   },
 });