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

Unified 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.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: static/js/index.js
===================================================================
--- a/static/js/index.js
+++ b/static/js/index.js
@@ -38,8 +38,8 @@
return;
var content = document.getElementById("content");
- var match = window.navigator.userAgent.match(/Edge\/(\d+.\d+)/);
- if (match && parseFloat(match[1]) >= 14.14342 &&
+ 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.
+ 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.
content.classList.contains("edge"))
content.classList.add("edge-supported");
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld