Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: static/js/index.js

Issue 29341378: Issue 4027 - make the download button available for windows insider users (Closed)
Patch Set: Created May 18, 2016, 12:13 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « static/img/edge_teaser.png ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 (function() 1 (function()
2 { 2 {
3 var visibleTab; 3 var visibleTab;
4 var container; 4 var container;
5 5
6 window.toggleMore = function() 6 window.toggleMore = function()
7 { 7 {
8 if (container.className == "hidden") 8 if (container.className == "hidden")
9 container.className = visibleTab || getDefaultTab(); 9 container.className = visibleTab || getDefaultTab();
10 else 10 else
(...skipping 11 matching lines...) Expand all
22 { 22 {
23 var content = document.getElementById("content"); 23 var content = document.getElementById("content");
24 var ua = content.className.match(/ua\-([^\s]+)/); 24 var ua = content.className.match(/ua\-([^\s]+)/);
25 visibleTab = ua && ua[1] || "firefox"; 25 visibleTab = ua && ua[1] || "firefox";
26 return visibleTab; 26 return visibleTab;
27 } 27 }
28 28
29 function init() 29 function init()
30 { 30 {
31 container = document.getElementById("more-container"); 31 container = document.getElementById("more-container");
32 checkEdgeSupport();
33 }
34
35 function checkEdgeSupport()
36 {
37 if (!window.navigator.userAgent)
38 return;
39
40 var content = document.getElementById("content");
41 var match = window.navigator.userAgent.match(/Edge\/(\d+.\d+)/);
42 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
Sebastian Noack 2016/05/18 12:57:25 Yeah, I agree. However, it seems, the change alrea
43 content.classList.contains("edge"))
44 content.classList.add("edge-supported");
32 } 45 }
33 46
34 init(); 47 init();
35 })(); 48 })();
OLDNEW
« no previous file with comments | « static/img/edge_teaser.png ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld