Allow different HardBreaks settings for documents and comments

custom/conf/app.ini.sample

@@ -216,7 +216,11 @@ EVENT_SOURCE_UPDATE_TIME = 10s
 ; Render soft line breaks as hard line breaks, which means a single newline character between
 ; paragraphs will cause a line break and adding trailing whitespace to paragraphs is not
 ; necessary to force a line break.
-ENABLE_HARD_LINE_BREAK = true
+; Previously one setting `ENABLE_HARD_LINE_BREAK` covered both
+; Render soft line breaks as hard line breaks for comments
+ENABLE_HARD_LINE_BREAK_IN_COMMENTS = true
+; Render soft line breaks as hard line breaks for markdown documents
+ENABLE_HARD_LINE_BREAK_IN_DOCUMENTS = false
 ; Comma separated list of custom URL-Schemes that are allowed as links when rendering Markdown
 ; for example git,magnet,ftp (more at https://en.wikipedia.org/wiki/List_of_URI_schemes)
 ; URLs starting with http and https are always displayed, whatever is put in this entry.

docs/content/doc/advanced/config-cheat-sheet.en-us.md

@@ -152,9 +152,12 @@ Values containing `#` or `;` must be quoted using `` ` `` or `"""`.
 
 ## Markdown (`markdown`)
 
-- `ENABLE_HARD_LINE_BREAK`: **true**: Render soft line breaks as hard line breaks, which
+- `ENABLE_HARD_LINE_BREAK_IN_COMMENTS`: **true**: Render soft line breaks as hard line breaks in comments, which
  means a single newline character between paragraphs will cause a line break and adding
- trailing whitespace to paragraphs is not necessary to force a line break.
+ trailing whitespace to paragraphs is not necessary to force a line break. (Previously one setting covered both.)
+- `ENABLE_HARD_LINE_BREAK_IN_DOCUMENTS`: **false**: Render soft line breaks as hard line breaks in documents, which
+ means a single newline character between paragraphs will cause a line break and adding
+ trailing whitespace to paragraphs is not necessary to force a line break. (Previously one setting covered both.)
 - `CUSTOM_URL_SCHEMES`: Use a comma separated list (ftp,git,svn) to indicate additional
  URL hyperlinks to be rendered in Markdown. URLs beginning in http and https are
  always displayed

models/repo.go

@@ -174,9 +174,10 @@ type Repository struct {
  *Mirror `xorm:"-"`
  Status RepositoryStatus `xorm:"NOT NULL DEFAULT 0"`
 
- RenderingMetas map[string]string `xorm:"-"`
- Units []*RepoUnit `xorm:"-"`
- PrimaryLanguage *LanguageStat `xorm:"-"`
+ RenderingMetas map[string]string `xorm:"-"`
+ DocumentRenderingMetas map[string]string `xorm:"-"`
+ Units []*RepoUnit `xorm:"-"`
+ PrimaryLanguage *LanguageStat `xorm:"-"`
 
  IsFork bool `xorm:"INDEX NOT NULL DEFAULT false"`
  ForkID int64 `xorm:"INDEX"`
@@ -534,11 +535,12 @@ func (repo *Repository) mustOwner(e Engine) *User {
 
 // ComposeMetas composes a map of metas for properly rendering issue links and external issue trackers.
 func (repo *Repository) ComposeMetas() map[string]string {
- if repo.RenderingMetas == nil {
+ if len(repo.RenderingMetas) == 0 {
  metas := map[string]string{
  "user": repo.OwnerName,
  "repo": repo.Name,
  "repoPath": repo.RepoPath(),
+ "mode": "comment",
  }
 
  unit, err := repo.GetUnit(UnitTypeExternalTracker)
@@ -570,6 +572,19 @@ func (repo *Repository) ComposeMetas() map[string]string {
  return repo.RenderingMetas
 }
 
+// ComposeDocumentMetas composes a map of metas for properly rendering documents
+func (repo *Repository) ComposeDocumentMetas() map[string]string {
+ if len(repo.DocumentRenderingMetas) == 0 {
+ metas := map[string]string{}
+ for k, v := range repo.ComposeMetas() {
+ metas[k] = v
+ }
+ metas["mode"] = "document"
+ repo.DocumentRenderingMetas = metas
+ }
+ return repo.DocumentRenderingMetas
+}
+
 // DeleteWiki removes the actual and local copy of repository wiki.
 func (repo *Repository) DeleteWiki() error {
  return repo.deleteWiki(x)

modules/markup/markdown/goldmark.go

@@ -151,6 +151,16 @@ func (g *ASTTransformer) Transform(node *ast.Document, reader text.Reader, pc pa
  v.AppendChild(v, newChild)
  }
  }
+ case *ast.Text:
+ if v.SoftLineBreak() && !v.HardLineBreak() {
+ renderMetas := pc.Get(renderMetasKey).(map[string]string)
+ mode := renderMetas["mode"]
+ if mode != "document" {
+ v.SetHardLineBreak(setting.Markdown.EnableHardLineBreakInComments)
+ } else {
+ v.SetHardLineBreak(setting.Markdown.EnableHardLineBreakInDocuments)
+ }
+ }
  }
  return ast.WalkContinue, nil
  })

modules/markup/markdown/markdown.go

@@ -29,17 +29,19 @@ var once = sync.Once{}
 
 var urlPrefixKey = parser.NewContextKey()
 var isWikiKey = parser.NewContextKey()
+var renderMetasKey = parser.NewContextKey()
 
 // NewGiteaParseContext creates a parser.Context with the gitea context set
-func NewGiteaParseContext(urlPrefix string, isWiki bool) parser.Context {
+func NewGiteaParseContext(urlPrefix string, metas map[string]string, isWiki bool) parser.Context {
  pc := parser.NewContext(parser.WithIDs(newPrefixedIDs()))
  pc.Set(urlPrefixKey, urlPrefix)
  pc.Set(isWikiKey, isWiki)
+ pc.Set(renderMetasKey, metas)
  return pc
 }
 
-// RenderRaw renders Markdown to HTML without handling special links.
-func RenderRaw(body []byte, urlPrefix string, wikiMarkdown bool) []byte {
+// render renders Markdown to HTML without handling special links.
+func render(body []byte, urlPrefix string, metas map[string]string, wikiMarkdown bool) []byte {
  once.Do(func() {
  converter = goldmark.New(
  goldmark.WithExtensions(extension.Table,
@@ -75,12 +77,9 @@ func RenderRaw(body []byte, urlPrefix string, wikiMarkdown bool) []byte {
  ),
  )
 
- if setting.Markdown.EnableHardLineBreak {
- converter.Renderer().AddOptions(html.WithHardWraps())
- }
  })
 
- pc := NewGiteaParseContext(urlPrefix, wikiMarkdown)
+ pc := NewGiteaParseContext(urlPrefix, metas, wikiMarkdown)
  var buf bytes.Buffer
  if err := converter.Convert(giteautil.NormalizeEOL(body), &buf, parser.WithContext(pc)); err != nil {
  log.Error("Unable to render: %v", err)
@@ -112,14 +111,19 @@ func (Parser) Extensions() []string {
 
 // Render implements markup.Parser
 func (Parser) Render(rawBytes []byte, urlPrefix string, metas map[string]string, isWiki bool) []byte {
- return RenderRaw(rawBytes, urlPrefix, isWiki)
+ return render(rawBytes, urlPrefix, metas, isWiki)
 }
 
 // Render renders Markdown to HTML with all specific handling stuff.
 func Render(rawBytes []byte, urlPrefix string, metas map[string]string) []byte {
  return markup.Render("a.md", rawBytes, urlPrefix, metas)
 }
 
+// RenderRaw renders Markdown to HTML without handling special links.
+func RenderRaw(body []byte, urlPrefix string, wikiMarkdown bool) []byte {
+ return render(body, urlPrefix, map[string]string{}, wikiMarkdown)
+}
+
 // RenderString renders Markdown to HTML with special links and returns string type.
 func RenderString(raw, urlPrefix string, metas map[string]string) string {
  return markup.RenderString("a.md", raw, urlPrefix, metas)

modules/setting/setting.go

@@ -256,12 +256,14 @@ var (
 
  // Markdown settings
  Markdown = struct {
- EnableHardLineBreak bool
- CustomURLSchemes []string `ini:"CUSTOM_URL_SCHEMES"`
- FileExtensions []string
+ EnableHardLineBreakInComments bool
+ EnableHardLineBreakInDocuments bool
+ CustomURLSchemes []string `ini:"CUSTOM_URL_SCHEMES"`
+ FileExtensions []string
  }{
- EnableHardLineBreak: true,
- FileExtensions: strings.Split(".md,.markdown,.mdown,.mkd", ","),
+ EnableHardLineBreakInComments: true,
+ EnableHardLineBreakInDocuments: false,
+ FileExtensions: strings.Split(".md,.markdown,.mdown,.mkd", ","),
  }
 
  // Admin settings

routers/api/v1/misc/markdown.go

@@ -48,6 +48,8 @@ func Markdown(ctx *context.APIContext, form api.MarkdownOption) {
  }
 
  switch form.Mode {
+ case "comment":
+ fallthrough
  case "gfm":
  md := []byte(form.Text)
  urlPrefix := form.Context
@@ -61,7 +63,14 @@ func Markdown(ctx *context.APIContext, form api.MarkdownOption) {
  }
  }
  if ctx.Repo != nil && ctx.Repo.Repository != nil {
- meta = ctx.Repo.Repository.ComposeMetas()
+ if form.Mode == "gfm" {
+ meta = ctx.Repo.Repository.ComposeDocumentMetas()
+ } else {
+ meta = ctx.Repo.Repository.ComposeMetas()
+ }
+ }
+ if form.Mode == "gfm" {
+ meta["mode"] = "document"
  }
  if form.Wiki {
  _, err := ctx.Write([]byte(markdown.RenderWiki(md, urlPrefix, meta)))

routers/repo/view.go

@@ -319,7 +319,7 @@ func renderDirectory(ctx *context.Context, treeLink string) {
  if markupType := markup.Type(readmeFile.name); markupType != "" {
  ctx.Data["IsMarkup"] = true
  ctx.Data["MarkupType"] = string(markupType)
- ctx.Data["FileContent"] = string(markup.Render(readmeFile.name, buf, readmeTreelink, ctx.Repo.Repository.ComposeMetas()))
+ ctx.Data["FileContent"] = string(markup.Render(readmeFile.name, buf, readmeTreelink, ctx.Repo.Repository.ComposeDocumentMetas()))
  } else {
  ctx.Data["IsRenderedHTML"] = true
  ctx.Data["FileContent"] = strings.Replace(
@@ -459,7 +459,7 @@ func renderFile(ctx *context.Context, entry *git.TreeEntry, treeLink, rawLink st
  if markupType := markup.Type(blob.Name()); markupType != "" {
  ctx.Data["IsMarkup"] = true
  ctx.Data["MarkupType"] = markupType
- ctx.Data["FileContent"] = string(markup.Render(blob.Name(), buf, path.Dir(treeLink), ctx.Repo.Repository.ComposeMetas()))
+ ctx.Data["FileContent"] = string(markup.Render(blob.Name(), buf, path.Dir(treeLink), ctx.Repo.Repository.ComposeDocumentMetas()))
  } else if readmeExist {
  ctx.Data["IsRenderedHTML"] = true
  ctx.Data["FileContent"] = strings.Replace(
@@ -538,7 +538,7 @@ func renderFile(ctx *context.Context, entry *git.TreeEntry, treeLink, rawLink st
  buf = append(buf, d...)
  ctx.Data["IsMarkup"] = true
  ctx.Data["MarkupType"] = markupType
- ctx.Data["FileContent"] = string(markup.Render(blob.Name(), buf, path.Dir(treeLink), ctx.Repo.Repository.ComposeMetas()))
+ ctx.Data["FileContent"] = string(markup.Render(blob.Name(), buf, path.Dir(treeLink), ctx.Repo.Repository.ComposeDocumentMetas()))
  }
 
  }

routers/repo/wiki.go

@@ -209,7 +209,7 @@ func renderViewPage(ctx *context.Context) (*git.Repository, *git.TreeEntry) {
  return nil, nil
  }
 
- metas := ctx.Repo.Repository.ComposeMetas()
+ metas := ctx.Repo.Repository.ComposeDocumentMetas()
  ctx.Data["content"] = markdown.RenderWiki(data, ctx.Repo.RepoLink, metas)
  ctx.Data["sidebarPresent"] = sidebarContent != nil
  ctx.Data["sidebarContent"] = markdown.RenderWiki(sidebarContent, ctx.Repo.RepoLink, metas)

templates/repo/editor/edit.tmpl

@@ -38,6 +38,7 @@
  <textarea id="edit_area" name="content" data-id="repo-{{.Repository.Name}}-{{.TreePath}}"
  data-url="{{.Repository.APIURL}}/markdown"
  data-context="{{.RepoLink}}"
+ data-markdown-mode="gfm"
  data-markdown-file-exts="{{.MarkdownFileExts}}"
  data-line-wrap-extensions="{{.LineWrapExtensions}}">
 {{.FileContent}}</textarea>

web_src/js/index.js

@@ -41,7 +41,7 @@ function initCommentPreviewTab($form) {
  const $this = $(this);
  $.post($this.data('url'), {
  _csrf: csrf,
- mode: 'gfm',
+ mode: 'comment',
  context: $this.data('context'),
  text: $form.find(`.tab[data-tab="${$tabMenu.data('write')}"] textarea`).val()
  }, (data) => {
@@ -65,14 +65,18 @@ function initEditPreviewTab($form) {
  $previewTab.on('click', function () {
  const $this = $(this);
  let context = `${$this.data('context')}/`;
+ let mode = this.data('markdown-mode');
+ if (!mode) {
+ mode = 'comment';
+ }
  const treePathEl = $form.find('input#tree_path');
  if (treePathEl.length > 0) {
  context += treePathEl.val();
  }
  context = context.substring(0, context.lastIndexOf('/'));
  $.post($this.data('url'), {
  _csrf: csrf,
- mode: 'gfm',
+ mode,
  context,
  text: $form.find(`.tab[data-tab="${$tabMenu.data('write')}"] textarea`).val()
  }, (data) => {