Merge pull request NLPchina#221 from stegano/patch-1

ansjsun · ansjsun · commit 3e771b7f27e1 · 2016-05-24T12:37:22.000+08:00
String crush issue on MS Excel
diff --git a/src/_site/controllers.js b/src/_site/controllers.js
@@ -255,7 +255,7 @@ function updateWithScrollIfNeeded (query) {
 		data += "\n";
 			data += map2csvStr(columns,rows[i],delimiter) ;
 		}
-		var plain = 'data:text/csv;charset=utf8,' + encodeURIComponent(data);
+		var plain = 'data:text/csv;charset=utf8,\uFEFF' + encodeURIComponent(data);
 		download(plain, "query_result.csv", "text/plain");
   		return true;
 	}
@@ -400,4 +400,4 @@ function updateWithScrollIfNeeded (query) {
 	function saveUrl() {
 		localStorage.setItem("lasturl", $scope.url);
 	}
-});
+});

Original file line number	Diff line number	Diff line change
`@@ -255,7 +255,7 @@ function updateWithScrollIfNeeded (query) {`
`255`	`255`	`data += "\n";`
`256`	`256`	`data += map2csvStr(columns,rows[i],delimiter) ;`
`257`	`257`	`}`
`258`		`- var plain = 'data:text/csv;charset=utf8,' + encodeURIComponent(data);`
	`258`	`+ var plain = 'data:text/csv;charset=utf8,\uFEFF' + encodeURIComponent(data);`
`259`	`259`	`download(plain, "query_result.csv", "text/plain");`
`260`	`260`	`return true;`
`261`	`261`	`}`
`@@ -400,4 +400,4 @@ function updateWithScrollIfNeeded (query) {`
`400`	`400`	`function saveUrl() {`
`401`	`401`	`localStorage.setItem("lasturl", $scope.url);`
`402`	`402`	`}`
`403`		`-});`
	`403`	`+});`