From c68e663b619aa3ccf08811b3cef047d14511e0b0 Mon Sep 17 00:00:00 2001
From: Prayag <55829513+prayag17@users.noreply.github.com>
Date: Sat, 26 Dec 2020 20:56:23 +0530
Subject: [PATCH] Change the selectors to Array like objects

---
 .../Configuration/configurationpage.html      | 82 ++++++++-----------
 1 file changed, 35 insertions(+), 47 deletions(-)

diff --git a/Jellyfin.Plugin.SkinManager/Configuration/configurationpage.html b/Jellyfin.Plugin.SkinManager/Configuration/configurationpage.html
index d028a04..e4a8bbd 100644
--- a/Jellyfin.Plugin.SkinManager/Configuration/configurationpage.html
+++ b/Jellyfin.Plugin.SkinManager/Configuration/configurationpage.html
@@ -382,48 +382,43 @@
                 Dashboard.processPluginConfigurationUpdateResult(res);});
             });
         }
-
         function getValue(option){
-          var result = "";
-          switch(option.type){
-           
-            case "checkBox":
-              document.getElementsByClassName("checkbox emby-checkbox").forEach(element =>{
-                if(option.css == element.getAttribute("data-css")){
-                  result =  element.checked?"true":"false";
-                  
-                }
-              })
-              
-            case "number":
-              document.getElementsByClassName("number emby-input").forEach(element =>{
-                  if(option.css == element.getAttribute("data-css")){
-                    result = element.value;
-                  }
-                })
-            case "colorPicker":
-              document.getElementsByClassName("color").forEach(element =>{
-                    if(option.css == element.getAttribute("data-css")){
-                      result = element.value;
-                    }
-                  })
-            case "slider":
-              document.getElementsByClassName("slider").forEach(element =>{
-                      if(option.css == element.getAttribute("data-css")){
-                        result = element.value;
-                      }
-                    })
-            case "selector":
-            document.getElementsByClassName("selector emby-select-withcolor emby-select").forEach(element =>{
-                      if(option.css == element.getAttribute("data-css")){
-                        result = element.selectedOptions[0].innerText;;
-                      }
-                    })
-             
-          }
-          return result;
-        }
+        var result = "";
+        switch(option.type){
+          case "checkBox":
+          Array.from(document.getElementsByClassName("checkbox emby-checkbox")).forEach(element =>{
+            if(option.css == element.getAttribute("data-css")){
+              result =  element.checked?"true":"false";
 
+            }
+          })
+          case "number":
+          Array.from(document.getElementsByClassName("number emby-input")).forEach(element =>{
+            if(option.css == element.getAttribute("data-css")){
+              result = element.value;
+            }
+          })
+          case "colorPicker":
+          Array.from(document.getElementsByClassName("color")).forEach(element =>{
+            if(option.css == element.getAttribute("data-css")){
+              result = element.value;
+            }
+          })
+          case "slider":
+          Array.from(document.getElementsByClassName("slider")).forEach(element =>{
+            if(option.css == element.getAttribute("data-css")){
+              result = element.value;
+            }
+          })
+          case "selector":
+          Array.from(document.getElementsByClassName("selector emby-select-withcolor emby-select")).forEach(element =>{
+            if(option.css == element.getAttribute("data-css")){
+              result = element.selectedOptions[0].innerText;;
+            }
+          })
+        }
+        return result;
+      }
         function checkEasterEggs(){
           var d = new Date();
 
@@ -441,14 +436,7 @@
               maxSize: 25
           });
         }); 
-        
         }
-
-
-        
-
-
-
         function setSkin() { 
           saveConfig()   
           ApiClient.getServerConfiguration().then(function (config) {