From 8dc81b1daa880a3a173577ec7d8c2a9c6e60191a Mon Sep 17 00:00:00 2001
From: NovapaX <maarten@oosterwijkbrandpreventie.nl>
Date: Mon, 4 Dec 2017 02:03:34 +0100
Subject: [PATCH] Color wheel fixes (#706)

* hide overflow properly

* only try to do rgb stuff on init if we have a value

* just to be sure
---
 src/components/ha-color-picker.html |  3 +++
 src/more-infos/more-info-light.html | 10 ++++++----
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/src/components/ha-color-picker.html b/src/components/ha-color-picker.html
index a92262a5ba..27b2bdfc3e 100644
--- a/src/components/ha-color-picker.html
+++ b/src/components/ha-color-picker.html
@@ -317,6 +317,9 @@ class HaColorPicker extends window.hassMixins.EventsMixin(Polymer.Element) {
   */
 
   applyRgbColor(rgb) {
+    if (!rgb) {
+      return;
+    }
     // do nothing is we already have the same color
     if (this.rgbColor &&
         this.rgbColor.r === rgb.r &&
diff --git a/src/more-infos/more-info-light.html b/src/more-infos/more-info-light.html
index ec753f794d..5cbd67d228 100644
--- a/src/more-infos/more-info-light.html
+++ b/src/more-infos/more-info-light.html
@@ -34,8 +34,8 @@
         width: 100%;
 
         max-height: 0px;
-        overflow: visible;
-        transition: max-height .2s ease-in;
+        overflow: hidden;
+        transition: max-height .5s ease-in;
       }
 
       .has-effect_list .effect_list,
@@ -47,6 +47,7 @@
 
       .has-rgb_color ha-color-picker {
         max-height: 500px;
+        overflow: visible;
         --ha-color-picker-wheel-borderwidth: 5;
         --ha-color-picker-wheel-bordercolor: white;
         --ha-color-picker-wheel-shadow: none;
@@ -159,8 +160,9 @@
         this.brightnessSliderValue = newVal.attributes.brightness;
         this.ctSliderValue = newVal.attributes.color_temp;
         this.wvSliderValue = newVal.attributes.white_value;
-        this.colorPickerColor = this.rgbArrToObj(newVal.attributes.rgb_color);
-
+        if (newVal.attributes.rgb_color) {
+          this.colorPickerColor = this.rgbArrToObj(newVal.attributes.rgb_color);
+        }
         if (newVal.attributes.effect_list) {
           this.effectIndex = newVal.attributes.effect_list.indexOf(newVal.attributes.effect);
         } else {