From 6a8e6a5a081415690bf89e87697d15b6ce9ebf8b Mon Sep 17 00:00:00 2001
From: Dennis Karpienski <dennis@karpienski.de>
Date: Sat, 7 May 2016 17:07:36 +0200
Subject: [PATCH] fixed select input (#51)

the event was fired too often, so the source was always reset
https://github.com/home-assistant/home-assistant/issues/1874
---
 src/more-infos/more-info-media_player.html | 2 +-
 src/more-infos/more-info-media_player.js   | 5 ++++-
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/more-infos/more-info-media_player.html b/src/more-infos/more-info-media_player.html
index 7c0a394e8d..0fc415e5e8 100644
--- a/src/more-infos/more-info-media_player.html
+++ b/src/more-infos/more-info-media_player.html
@@ -73,7 +73,7 @@
           on-change='volumeSliderChanged' class='flex'>
         </paper-slider>
       </div>
-      <div class='controls layout horizontal justified' hidden$='[[!supportsSelectSource]]'>
+      <div class='controls layout horizontal justified' hidden$='[[!computeHideSelectSource(isOff, supportsSelectSource)]]'>
           <iron-icon class="source-input" icon="mdi:login-variant"></iron-icon>
           <paper-dropdown-menu class="source-input" label-float label='Source'>
             <paper-menu class="dropdown-content" selected="{{sourceIndex}}">
diff --git a/src/more-infos/more-info-media_player.js b/src/more-infos/more-info-media_player.js
index f43fef9ce8..8234bce024 100644
--- a/src/more-infos/more-info-media_player.js
+++ b/src/more-infos/more-info-media_player.js
@@ -108,7 +108,6 @@ export default new Polymer({
       this.volumeSliderValue = newVal.attributes.volume_level * 100;
       this.isMuted = newVal.attributes.is_volume_muted;
       this.source = newVal.attributes.source;
-      this.sourceIndex = 0;
       this.supportsPause = (newVal.attributes.supported_media_commands & 1) !== 0;
       this.supportsVolumeSet = (newVal.attributes.supported_media_commands & 4) !== 0;
       this.supportsVolumeMute = (newVal.attributes.supported_media_commands & 8) !== 0;
@@ -158,6 +157,10 @@ export default new Polymer({
     return isOff ? !supportsTurnOn : !supportsTurnOff;
   },
 
+  computeHideSelectSource(isOff, supportsSelectSource) {
+    return !isOff && supportsSelectSource;
+  },
+
   computeSelectedSource(stateObj) {
     return stateObj.attributes.source_list.indexOf(stateObj.attributes.source);
   },