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");
-    if(content.className.indexOf("edge") == -1 || !window.navigator.userAgent)

[Sebastian Noack, 2016/05/17 17:02:11]

If you put the check here below the UA check, it will be safe to use classList
here as well.

[Sebastian Noack, 2016/05/17 17:02:11]

Nit: Missing space after "if".

[saroyanm, 2016/05/18 12:03:41]

Done.

[saroyanm, 2016/05/18 12:03:41]

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)
+    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();
 })();