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

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

Issue 29342791: Issue 4027 - Check only for windows build number (Closed)
Patch Set: Created May 18, 2016, 1:11 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 | « no previous file | 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 20 matching lines...) Expand all
31 container = document.getElementById("more-container"); 31 container = document.getElementById("more-container");
32 checkEdgeSupport(); 32 checkEdgeSupport();
33 } 33 }
34 34
35 function checkEdgeSupport() 35 function checkEdgeSupport()
36 { 36 {
37 if (!window.navigator.userAgent) 37 if (!window.navigator.userAgent)
38 return; 38 return;
39 39
40 var content = document.getElementById("content"); 40 var content = document.getElementById("content");
41 var match = window.navigator.userAgent.match(/Edge\/(\d+.\d+)/); 41 var match = window.navigator.userAgent.match(/Edge\/\d+.(\d+)/);
Sebastian Noack 2016/05/18 13:15:31 Apparently this issue isn't new with this change,
saroyanm 2016/05/18 13:18:49 Done.
42 if (match && parseFloat(match[1]) >= 14.14342 && 42 if (match && parseFloat(match[1]) >= 14342 &&
Sebastian Noack 2016/05/18 13:15:31 It seems parseInt() is more appropriate now. Or si
saroyanm 2016/05/18 13:18:49 Done.
43 content.classList.contains("edge")) 43 content.classList.contains("edge"))
44 content.classList.add("edge-supported"); 44 content.classList.add("edge-supported");
45 } 45 }
46 46
47 init(); 47 init();
48 })(); 48 })();
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld