]> source.dussan.org Git - gitea.git/commitdiff
Fix line spacing for plaintext previews (#22699)
authorcrystal <71373843+CrystalCommunication@users.noreply.github.com>
Thu, 2 Feb 2023 04:51:02 +0000 (21:51 -0700)
committerGitHub <noreply@github.com>
Thu, 2 Feb 2023 04:51:02 +0000 (22:51 -0600)
Adding `<br>` between each line is not necessary since the entire file
is rendered inside a `<pre>`

fixes https://codeberg.org/Codeberg/Community/issues/915

modules/charset/escape.go

index 3b1c20697793b11cf548abe9e133770d87d7cabd..5608836a4510e0eb7c85bbdad0d0ff8a61b6db0c 100644 (file)
@@ -44,7 +44,7 @@ func EscapeControlReader(reader io.Reader, writer io.Writer, locale translation.
        return streamer.escaped, err
 }
 
-// EscapeControlStringReader escapes the unicode control sequences in a provided reader of string content and writer in a locale and returns the findings as an EscapeStatus and the escaped []byte
+// EscapeControlStringReader escapes the unicode control sequences in a provided reader of string content and writer in a locale and returns the findings as an EscapeStatus and the escaped []byte. HTML line breaks are not inserted after every newline by this method.
 func EscapeControlStringReader(reader io.Reader, writer io.Writer, locale translation.Locale, allowed ...rune) (escaped *EscapeStatus, err error) {
        bufRd := bufio.NewReader(reader)
        outputStream := &HTMLStreamerWriter{Writer: writer}
@@ -65,10 +65,6 @@ func EscapeControlStringReader(reader io.Reader, writer io.Writer, locale transl
                        }
                        break
                }
-               if err := streamer.SelfClosingTag("br"); err != nil {
-                       streamer.escaped.HasError = true
-                       return streamer.escaped, err
-               }
        }
        return streamer.escaped, err
 }