Relocated icon and redesigned icon popup

stats.js

 /*
  * This file is part of Adblock Plus <http://adblockplus.org/>,
  * Copyright (C) 2006-2013 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
 (function()
 {
   var backgroundPage = ext.backgroundPage.getWindow();
   var require = backgroundPage.require;
   var getStats = require("stats").getStats;
   var FilterNotifier = require("filterNotifier").FilterNotifier;
+  var Prefs = require("prefs").Prefs;
   
   var currentTab;
   var shareURL = "https://adblockplus.org/";
   
   var messageMark = {};
   var shareLinks = {
     facebook: ["https://www.facebook.com/dialog/feed", {
       app_id: "475542399197328",
       link: shareURL,
       redirect_uri: "https://www.facebook.com/",
@@ skipping 27 matching lines @@
       var value = params[key];
       if (value == messageMark)
         value = i18n.getMessage("stats_share_message", blockedCount);
       querystring.push(encodeURIComponent(key) + "=" + encodeURIComponent(value));
     }
     return url + "?" + querystring.join("&");
   }
   
   function onLoad()
   {
-    document.getElementById("shareBox").addEventListener("click", share, false);
-    document.getElementById("share").addEventListener("click", toggleShareBox, false);
+    document.getElementById("share-box").addEventListener("click", share, false);
+    var showIconNumber = document.getElementById("show-iconnumber");
+    showIconNumber.setAttribute("aria-checked", Prefs.show_iconnumber);

[Felix Dahlke, 2013/12/02 15:45:58]

What does "aria" mean here?

[Thomas Greiner, 2013/12/03 12:06:05]

It's a standard set of attributes to make custom elements more accessible (e.g.
label a div as a button which acts as one).

I know that we're currently not using this standard but I thought in this case -
since I needed a "checked" attribute anyway - it might be good to use
"aria-checked" instead of "data-checked".

More information:
(1) https://developer.mozilla.org/en-US/docs/Accessibility/ARIA
(2) http://www.w3.org/TR/wai-aria/states_and_properties

[Wladimir Palant, 2013/12/03 12:55:43]

That's mostly because we are usually using standard widget where we don't have
to identify the role or state explicitly.

[Felix Dahlke, 2013/12/04 12:30:06]

Ah, I see. Well I would definitely prefer using standard controls, but this aria
stuff seems like the next best thing.

+    showIconNumber.addEventListener("click", toggleIconNumber, false);
+    document.querySelector("label[for='show-iconnumber']").addEventListener("click", toggleIconNumber, false);
     
     // Update stats
     ext.windows.getLastFocused(function(win)
     {
       win.getActiveTab(function(tab)
       {
         currentTab = tab;
         updateStats();
 
         FilterNotifier.addListener(onNotify);
 
-        document.getElementById("statsContainer").removeAttribute("hidden");
+        document.getElementById("stats-container").removeAttribute("hidden");
       });
     });
   }
   
   function onUnload()
   {
     FilterNotifier.removeListener(onNotify);
   }
   
   function onNotify(action, item)
   {
     if (action == "filter.hitCount")
       updateStats();
   }
   
   function updateStats()
   {
-    var statsPage = document.getElementById("statsPage");
+    var statsPage = document.getElementById("stats-page");
     var blockedPage = getStats("blocked", currentTab).toLocaleString();
     i18n.setElementText(statsPage, "stats_label_page", [blockedPage]);
     
-    var statsTotal = document.getElementById("statsTotal");
+    var statsTotal = document.getElementById("stats-total");
     var blockedTotal = getStats("blocked").toLocaleString();
     i18n.setElementText(statsTotal, "stats_label_total", [blockedTotal]);
   }
   
-  function toggleShareBox(ev)
-  {
-    var shareBox = document.getElementById("shareBox");
-    shareBox.hidden = !shareBox.hidden;
-  }
-  
   function share(ev)
   {
     // Easter Egg
     var blocked = getStats("blocked");
     if (blocked <= 9000 || blocked >= 10000)
       blocked = blocked.toLocaleString();
     else
       blocked = i18n.getMessage("stats_over", (9000).toLocaleString());
     
     var url = createShareLink(ev.target.dataset.social, blocked);
     ext.windows.getLastFocused(function(win) { win.openTab(url); });
   }
   
+  function toggleIconNumber()
+  {
+    Prefs.show_iconnumber = !Prefs.show_iconnumber;
+    document.getElementById("show-iconnumber").setAttribute("aria-checked", Prefs.show_iconnumber);
+  }
+  
   document.addEventListener("DOMContentLoaded", onLoad, false);
   window.addEventListener("unload", onUnload, false);
 })();