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

Delta Between Two Patch Sets: static/js/index.js

Issue 29342791: Issue 4027 - Check only for windows build number (Closed)
Left Patch Set: Created May 18, 2016, 1:11 p.m.
Right Patch Set: Created May 18, 2016, 1:18 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « no previous file | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
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]) >= 14342 && 42 if (match && match[1] >= 14342 && content.classList.contains("edge"))
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"))
44 content.classList.add("edge-supported"); 43 content.classList.add("edge-supported");
45 } 44 }
46 45
47 init(); 46 init();
48 })(); 47 })();
LEFTRIGHT
« no previous file | no next file » | Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Toggle Comments ('s')

Powered by Google App Engine
This is Rietveld