Merge pull request #273 from georgejhunt/ie11

tim-moody · web-flow · commit d33743b3acf9 · 2019-12-19T11:17:17.000-05:00
IE 11 does not have string.includes()
diff --git a/roles/js-menu/files/menu-files/js/js-menu.js b/roles/js-menu/files/menu-files/js/js-menu.js
@@ -228,7 +228,7 @@ function procStatic(){
 }
 
 function procMenu() {
-	if (isMobile || navigator.userAgent.includes('Win64') || navigator.platform == 'MacIntel') {
+	if (isMobile || navigator.userAgent.search('Win64') !== -1 || navigator.platform == 'MacIntel') {
     	var allowed = menuConfig.apache_allow_sudo || false;
     	var desired = menuParams.allow_server_time_update || false;
   	  if (allowed && desired)

Original file line number	Diff line number	Diff line change
`@@ -228,7 +228,7 @@ function procStatic(){`
`228`	`228`	`}`
`229`	`229`
`230`	`230`	`function procMenu() {`
`231`		`- if (isMobile \|\| navigator.userAgent.includes('Win64') \|\| navigator.platform == 'MacIntel') {`
	`231`	`+ if (isMobile \|\| navigator.userAgent.search('Win64') !== -1 \|\| navigator.platform == 'MacIntel') {`
`232`	`232`	`var allowed = menuConfig.apache_allow_sudo \|\| false;`
`233`	`233`	`var desired = menuParams.allow_server_time_update \|\| false;`
`234`	`234`	`if (allowed && desired)`