Skip to content

Commit 37ed92d

Browse files
authored
Revert "Fix EOL handling in web editor" (#28101)
Reverts #27141 close #28097
1 parent 4d0eba8 commit 37ed92d

File tree

3 files changed

+6
-22
lines changed

3 files changed

+6
-22
lines changed

routers/web/repo/editor.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -287,7 +287,7 @@ func editFilePost(ctx *context.Context, form forms.EditRepoFileForm, isNewFile b
287287
Operation: operation,
288288
FromTreePath: ctx.Repo.TreePath,
289289
TreePath: form.TreePath,
290-
ContentReader: strings.NewReader(form.Content),
290+
ContentReader: strings.NewReader(strings.ReplaceAll(form.Content, "\r", "")),
291291
},
292292
},
293293
Signoff: form.Signoff,

templates/repo/editor/edit.tmpl

+3-4
Original file line numberDiff line numberDiff line change
@@ -34,13 +34,12 @@
3434
{{end}}
3535
</div>
3636
<div class="ui bottom attached active tab segment" data-tab="write">
37-
<textarea id="edit_area" name="content" class="gt-hidden"
38-
data-id="repo-{{.Repository.Name}}-{{.TreePath}}"
37+
<textarea id="edit_area" name="content" class="gt-hidden" data-id="repo-{{.Repository.Name}}-{{.TreePath}}"
3938
data-url="{{.Repository.Link}}/markup"
4039
data-context="{{.RepoLink}}"
4140
data-previewable-extensions="{{.PreviewableExtensions}}"
42-
data-line-wrap-extensions="{{.LineWrapExtensions}}"
43-
data-initial-value="{{JsonUtils.EncodeToString .FileContent}}"></textarea>
41+
data-line-wrap-extensions="{{.LineWrapExtensions}}">
42+
{{.FileContent}}</textarea>
4443
<div class="editor-loading is-loading"></div>
4544
</div>
4645
<div class="ui bottom attached tab segment markup" data-tab="preview">

web_src/js/features/codeeditor.js

+2-17
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ export async function createMonaco(textarea, filename, editorOpts) {
6262
const monaco = await import(/* webpackChunkName: "monaco" */'monaco-editor');
6363

6464
initLanguages(monaco);
65-
let {language, eol, ...other} = editorOpts;
65+
let {language, ...other} = editorOpts;
6666
if (!language) language = getLanguage(filename);
6767

6868
const container = document.createElement('div');
@@ -105,28 +105,14 @@ export async function createMonaco(textarea, filename, editorOpts) {
105105
monaco.languages.register({id: 'vs.editor.nullLanguage'});
106106
monaco.languages.setLanguageConfiguration('vs.editor.nullLanguage', {});
107107

108-
// We encode the initial value in JSON on the backend to prevent browsers from
109-
// discarding the \r during HTML parsing:
110-
// https://html.spec.whatwg.org/multipage/parsing.html#preprocessing-the-input-stream
111-
const value = JSON.parse(textarea.getAttribute('data-initial-value') || '""');
112-
textarea.value = value;
113-
textarea.removeAttribute('data-initial-value');
114-
115108
const editor = monaco.editor.create(container, {
116-
value,
109+
value: textarea.value,
117110
theme: 'gitea',
118111
language,
119112
...other,
120113
});
121114

122115
const model = editor.getModel();
123-
124-
// Monaco performs auto-detection of dominant EOL in the file, biased towards LF for
125-
// empty files. If there is an editorconfig value, override this detected value.
126-
if (eol in monaco.editor.EndOfLineSequence) {
127-
model.setEOL(monaco.editor.EndOfLineSequence[eol]);
128-
}
129-
130116
model.onDidChangeContent(() => {
131117
textarea.value = editor.getValue();
132118
textarea.dispatchEvent(new Event('change')); // seems to be needed for jquery-are-you-sure
@@ -201,6 +187,5 @@ function getEditorConfigOptions(ec) {
201187
opts.trimAutoWhitespace = ec.trim_trailing_whitespace === true;
202188
opts.insertSpaces = ec.indent_style === 'space';
203189
opts.useTabStops = ec.indent_style === 'tab';
204-
opts.eol = ec.end_of_line?.toUpperCase();
205190
return opts;
206191
}

0 commit comments

Comments
 (0)