Support CRLF when splitting code lines for display

Author: silverwind

public/css/index.css

@@ -1435,7 +1435,7 @@ footer .ui.language .menu {
 .repository.file.list #file-content .code-view .lines-code ol li,
 .repository.file.list #file-content .code-view .lines-num .hljs li,
 .repository.file.list #file-content .code-view .lines-code .hljs li {
-  display: inline-block;
+  display: block;
   width: 100%;
 }
 .repository.file.list #file-content .code-view .lines-num pre li.active,

public/less/_repository.less

@@ -296,7 +296,7 @@
 						margin: 0;
 						padding: 0 !important;
 						li {
-							display: inline-block;
+							display: block;
 							width: 100%;
 							&.active {
 								background: #ffffdd;

routers/repo/view.go

@@ -11,6 +11,7 @@ import (
 	gotemplate "html/template"
 	"io/ioutil"
 	"path"
+	"regexp"
 	"strconv"
 	"strings"
 
@@ -210,10 +211,26 @@ func renderFile(ctx *context.Context, entry *git.TreeEntry, treeLink, rawLink st
 			}
 
 			var output bytes.Buffer
-			lines := strings.Split(fileContent, "\n")
+			var terminator string
+
+			// test for bare linefeeds in case of mixed terminators
+			lf, _ := regexp.MatchString("[^\r]\n", str)
+
+			if strings.Index(fileContent, "\r\n") != -1 && !lf {
+				terminator = "\r\n"
+			} else {
+				terminator = "\n"
+			}
+			lines := strings.Split(fileContent, terminator)
+
 			for index, line := range lines {
-				output.WriteString(fmt.Sprintf(`<li class="L%d" rel="L%d">%s</li>`, index+1, index+1, gotemplate.HTMLEscapeString(line)) + "\n")
+				line = gotemplate.HTMLEscapeString(line)
+				if index != len(lines) - 1 {
+					line += terminator
+				}
+				output.WriteString(fmt.Sprintf(`<li class="L%d" rel="L%d">%s</li>`, index+1, index+1, line))
 			}
+
 			ctx.Data["FileContent"] = gotemplate.HTML(output.String())
 
 			output.Reset()