Issue 2383 - Added links to options page sidebar

options.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 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
@@ skipping 307 matching lines @@
 
   function onDOMLoaded()
   {
     var recommendationTemplate = document.querySelector("#recommend-list-table template");
     var popularText = ext.i18n.getMessage("options_popular");
     recommendationTemplate.content.querySelector(".popular").textContent = popularText;
     var languagesTemplate = document.querySelector("#all-lang-table template");
     var buttonText = ext.i18n.getMessage("options_button_add");
     languagesTemplate.content.querySelector(".button-add span").textContent = buttonText;
 
-    updateShareLink();
     populateLists();
 
     var tabList = document.querySelectorAll("#main-navigation-tabs li");
     for (var i = 0; i < tabList.length; i++)
     {
       tabList[i].addEventListener("click", function(e)
       {
         document.body.dataset.tab = e.currentTarget.dataset.show;
       }, false);
     }
 
     function onFindLanguageKeyUp()
     {
       var searchStyle = E("search-style");
       if (!this.value)
         searchStyle.innerHTML = "";
       else
         searchStyle.innerHTML = "#all-lang-table li:not([data-search*=\"" + this.value.toLowerCase() + "\"]) { display: none; }";
     }
 
-    // Update version number in navigation sidebar
+    // Initialize navigation sidebar
     ext.backgroundPage.sendMessage(
     {
-      method: "app.get",
+      type: "app.get",
       what: "addonVersion"
     },
     function(addonVersion)
     {
       E("abp-version").textContent = addonVersion;
     });
-    
+    getDocLink("releases", function(link)
+    {
+      E("link-version").setAttribute("href", link);
+    });
+
+    getDocLink("contribute", function(link)
+    {
+      document.querySelector("#tab-contribute a").setAttribute("href", link);
+    });
+
+    updateShareLink();
+
+    // Initialize interactive UI elements
     var placeholderValue  = ext.i18n.getMessage("options_dialog_language_find");
     E("find-language").setAttribute("placeholder", placeholderValue);
     E("add-blocking-list").addEventListener("click", function()
     {
       openDialog("customlist");
     }, false);
     E("add-website-language").addEventListener("click", function()
     {
       openDialog("language");
     }, false);
@@ skipping 243 matching lines @@
     {
       // TODO: modify "share" link accordingly
     });
   }
 
   function E(id)
   {
     return document.getElementById(id);
   }
 
+  function getDocLink(link, callback)
+  {
+    ext.backgroundPage.sendMessage({

[saroyanm, 2015/06/29 15:59:17]

nit: please place the opening curly bracket in the new line for consistency.

[Thomas Greiner, 2015/06/29 16:34:11]

I changed it but seeing that it's done differently in firstRun.js, I'd actually
recommend switching to the syntax from there to at least ensure consistency
within the adblockplusui repository.

[saroyanm, 2015/06/29 18:40:41]

Good point, will create ticket for that.

+      type: "app.get",
+      what: "doclink",
+      link: link
+    }, callback);
+  }
+
   ext.onMessage.addListener(function(message)
   {
     switch (message.type)
     {
       case "app.listen":
         if (message.action == "addSubscription")
           showAddSubscriptionDialog(message.args[0]);
         break;
       case "filters.listen":
         onFilterMessage(message.action, message.args[0]);
@@ skipping 15 matching lines @@
     filter: ["added", "loaded", "removed"]
   });
   ext.backgroundPage.sendMessage(
   {
     type: "subscriptions.listen",
     filter: ["added", "disabled", "homepage", "removed", "title"]
   });
 
   window.addEventListener("DOMContentLoaded", onDOMLoaded, false);
 })();