From fcd06a9000abf707c22b4684251d8c50f3e67f90 Mon Sep 17 00:00:00 2001
From: Yosi Levy <yosilevy@gmail.com>
Date: Sat, 16 Feb 2019 22:28:46 +0200
Subject: [PATCH] Updated super + commets

---
 src/components/ha-icon-next.ts                    | 1 +
 src/components/ha-icon-prev.ts                    | 1 +
 src/components/ha-paper-icon-button-arrow-next.ts | 5 +++--
 src/components/ha-paper-icon-button-arrow-prev.ts | 5 +++--
 src/components/ha-paper-icon-button-next.ts       | 5 +++--
 src/components/ha-paper-icon-button-prev.ts       | 5 +++--
 6 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/src/components/ha-icon-next.ts b/src/components/ha-icon-next.ts
index 369d5fc43b..d6a16bc7bb 100644
--- a/src/components/ha-icon-next.ts
+++ b/src/components/ha-icon-next.ts
@@ -10,6 +10,7 @@ export class HaIconNext extends HaIcon {
         ? "hass:chevron-right"
         : "hass:chevron-left";
 
+    // calling super after setting icon to have it consistently show the icon (otherwise not always shown)
     super.connectedCallback();
   }
 }
diff --git a/src/components/ha-icon-prev.ts b/src/components/ha-icon-prev.ts
index 924c0d1fed..8fa3c030e8 100644
--- a/src/components/ha-icon-prev.ts
+++ b/src/components/ha-icon-prev.ts
@@ -10,6 +10,7 @@ export class HaIconPrev extends HaIcon {
         ? "hass:chevron-left"
         : "hass:chevron-right";
 
+    // calling super after setting icon to have it consistently show the icon (otherwise not always shown)
     super.connectedCallback();
   }
 }
diff --git a/src/components/ha-paper-icon-button-arrow-next.ts b/src/components/ha-paper-icon-button-arrow-next.ts
index 2a399600ca..8d0914a0cb 100644
--- a/src/components/ha-paper-icon-button-arrow-next.ts
+++ b/src/components/ha-paper-icon-button-arrow-next.ts
@@ -10,12 +10,13 @@ const paperIconButtonClass = customElements.get(
 
 export class HaPaperIconButtonArrowNext extends paperIconButtonClass {
   public connectedCallback() {
-    super.connectedCallback();
-
     this.icon =
       window.getComputedStyle(this).direction === "ltr"
         ? "hass:arrow-right"
         : "hass:arrow-left";
+
+    // calling super after setting icon to have it consistently show the icon (otherwise not always shown)
+    super.connectedCallback();
   }
 }
 
diff --git a/src/components/ha-paper-icon-button-arrow-prev.ts b/src/components/ha-paper-icon-button-arrow-prev.ts
index d1cd7a0fd0..acac478b8f 100644
--- a/src/components/ha-paper-icon-button-arrow-prev.ts
+++ b/src/components/ha-paper-icon-button-arrow-prev.ts
@@ -10,12 +10,13 @@ const paperIconButtonClass = customElements.get(
 
 export class HaPaperIconButtonArrowPrev extends paperIconButtonClass {
   public connectedCallback() {
-    super.connectedCallback();
-
     this.icon =
       window.getComputedStyle(this).direction === "ltr"
         ? "hass:arrow-left"
         : "hass:arrow-right";
+
+    // calling super after setting icon to have it consistently show the icon (otherwise not always shown)
+    super.connectedCallback();
   }
 }
 
diff --git a/src/components/ha-paper-icon-button-next.ts b/src/components/ha-paper-icon-button-next.ts
index 28ca138277..5a137fc2d3 100644
--- a/src/components/ha-paper-icon-button-next.ts
+++ b/src/components/ha-paper-icon-button-next.ts
@@ -10,12 +10,13 @@ const paperIconButtonClass = customElements.get(
 
 export class HaPaperIconButtonNext extends paperIconButtonClass {
   public connectedCallback() {
-    super.connectedCallback();
-
     this.icon =
       window.getComputedStyle(this).direction === "ltr"
         ? "hass:chevron-right"
         : "hass:chevron-left";
+
+    // calling super after setting icon to have it consistently show the icon (otherwise not always shown)
+    super.connectedCallback();
   }
 }
 
diff --git a/src/components/ha-paper-icon-button-prev.ts b/src/components/ha-paper-icon-button-prev.ts
index 88b84bf621..06479fcd1f 100644
--- a/src/components/ha-paper-icon-button-prev.ts
+++ b/src/components/ha-paper-icon-button-prev.ts
@@ -10,12 +10,13 @@ const paperIconButtonClass = customElements.get(
 
 export class HaPaperIconButtonPrev extends paperIconButtonClass {
   public connectedCallback() {
-    super.connectedCallback();
-
     this.icon =
       window.getComputedStyle(this).direction === "ltr"
         ? "hass:chevron-left"
         : "hass:chevron-right";
+
+    // calling super after setting icon to have it consistently show the icon (otherwise not always shown)
+    super.connectedCallback();
   }
 }