Issue 4027 - make the download button available for windows insider users

static/js/index.js

 (function()
 {
   var visibleTab;
   var container;
 
   window.toggleMore = function()
   {
     if (container.className == "hidden")
       container.className = visibleTab || getDefaultTab();
     else
@@ skipping 16 matching lines @@
   }
 
   function init()
   {
     container = document.getElementById("more-container");
     checkEdgeSupport();
   }
 
   function checkEdgeSupport()
   {
-    var content = document.getElementById("content");

[Sebastian Noack, 2016/05/18 12:06:38]

We always assign variables where they are (first) used.

[saroyanm, 2016/05/18 12:14:10]

Done.

     if (!window.navigator.userAgent)
       return;
 
+    var content = document.getElementById("content");
     var match = window.navigator.userAgent.match(/Edge\/(\d+.\d+)/);
     if (match && parseFloat(match[1]) >= 14.14342 &&

[Oleksandr, 2016/05/18 12:53:15]

14 is a version of an EdgeHTML. 14342 is a Windows build number. Logically I
think we should check only the Windows build number. EdgeHTML version might not
be connected to the extensions functionality at all.

See:
https://blogs.windows.com/msedgedev/2015/09/21/understanding-versions-in-an-e...

[Sebastian Noack, 2016/05/18 12:57:25]

Yeah, I agree. However, it seems, the change already landed.

         content.classList.contains("edge"))
       content.classList.add("edge-supported");
   }
 
   init();
 })();