Fixed button icon state displaying

Signed-off-by: Hofi <hofione@gmail.com>

Author: HofiOne

_includes/skins.html

@@ -76,8 +76,8 @@
       setCookie(selectedSkinBackgroundColorKey, backgroundColor);
     };    
 
-    function toggleIcon(target, off) {
-      if (off) {
+    function setIcon(target, isOff) {
+      if (isOff) {
         target.classList.remove('fa-toggle-on');
         target.classList.add('fa-toggle-off');
       }
@@ -91,7 +91,7 @@
       var off = getSkinModeState() == darkModeStateOff;
 
       setSkinModeState(off ? darkModeStateOn : darkModeStateOff);
-      toggleIcon(event.target, off);
+      setIcon(event.target, off == false);
 
       event.currentTarget.blur();
     }
@@ -107,8 +107,10 @@
       // Why this is not working?!?!
       //document.body.style.removeProperty("backgroundColor");
       document.body.style.backgroundColor = "";
-      
-      toggleIcon($('#skin-button').find('.masthead_button_icon')[0], getSkinModeState());
+
+      var currSkinModeState = getSkinModeState();
+      setIcon($('#skin-button').find('.masthead_button_icon')[0], currSkinModeState == darkModeStateOff);
+
       $("#skin-button").on("click", toggleSkin);
 
       // TODO: See, setSkinModeState - stylesheet.onload