From d7071a81bde4e3b928af4b85cc51c42a623ae0ae Mon Sep 17 00:00:00 2001
From: Gabriele <pmzqla.git@gmail.com>
Date: Sun, 30 Nov 2014 21:13:02 +0100
Subject: [PATCH] WebUI: Keep current preference for the title style in a
 variable

Read persistent value only when required.
---
 src/webui/www/public/scripts/client.js | 26 ++++++++++++--------------
 1 file changed, 12 insertions(+), 14 deletions(-)

diff --git a/src/webui/www/public/scripts/client.js b/src/webui/www/public/scripts/client.js
index ba4b6d2bf..83a47b28d 100644
--- a/src/webui/www/public/scripts/client.js
+++ b/src/webui/www/public/scripts/client.js
@@ -75,6 +75,11 @@ window.addEvent('load', function () {
         height : 300
     });
     initializeWindows();
+
+    var speedInTitle = localStorage.getItem('speed_in_browser_title_bar') == "true";
+    if (!speedInTitle)
+        $('speedInBrowserTitleBarLink').firstChild.style.opacity = '0';
+
     var r = 0;
 
     var stateToImg = function (state) {
@@ -111,7 +116,7 @@ window.addEvent('load', function () {
                     $("UpInfos").set('html', "_(U: %1 - T: %2)"
                         .replace("%1", friendlyUnit(info.up_info_speed, true))
                         .replace("%2", friendlyUnit(info.up_info_data, false)));
-                    if(localStorage.getItem('speed_in_browser_title_bar') == 'true')
+                    if (speedInTitle)
                         document.title = "_(D:%1 U:%2)".replace("%1", friendlyUnit(info.dl_info_speed, true)).replace("%2", friendlyUnit(info.up_info_speed, true));
                     else
                         document.title = "_(qBittorrent web User Interface)";
@@ -233,20 +238,13 @@ window.addEvent('load', function () {
         updateTransferList();
     };
 
-    updateSpeedInBrowserTitleBarLinkCheckState = function() {
-        if (localStorage.getItem('speed_in_browser_title_bar') == 'true')
-            $(speedInBrowserTitleBarLink).firstChild.style.opacity = '1';
+    $('speedInBrowserTitleBarLink').addEvent('click', function(e) {
+        speedInTitle = !speedInTitle;
+        localStorage.setItem('speed_in_browser_title_bar', speedInTitle.toString());
+        if (speedInTitle)
+            $('speedInBrowserTitleBarLink').firstChild.style.opacity = '1';
         else
-            $(speedInBrowserTitleBarLink).firstChild.style.opacity = '0';
-    }
-
-    updateSpeedInBrowserTitleBarLinkCheckState();
-
-    addClickEvent('speedInBrowserTitleBar', function() {
-        var speed_in_browser_title_bar = localStorage.getItem('speed_in_browser_title_bar');
-        speed_in_browser_title_bar = speed_in_browser_title_bar == 'true' ? 'false' : 'true';
-        localStorage.setItem('speed_in_browser_title_bar', speed_in_browser_title_bar);
-        updateSpeedInBrowserTitleBarLinkCheckState();
+            $('speedInBrowserTitleBarLink').firstChild.style.opacity = '0';
         updateTransferInfo();
     });