thepaperpilot/forgejo
1
0
Fork 0
Code Issues Pull requests Releases Activity Actions

fix hljs unintenionally highlighting commit links (#7244)

Browse source 
* fix hljs unintenionally highlighting commit links

* fix unit tests
This commit is contained in:
silverwind 2019-06-19 00:31:31 +02:00 committed by techknowlogick
parent a205155e9d
commit b209531959
2 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
modules/markup/html.go
View file

@ -323,6 +323,7 @@ func createCodeLink(href, content string) *html.Node {
code := &html.Node{ code := &html.Node{
Type: html.ElementNode, Type: html.ElementNode,
Data: atom.Code.String(), Data: atom.Code.String(),
Attr: []html.Attribute{{Key: "class", Val: "nohighlight"}},
} }
code.AppendChild(text) code.AppendChild(text)

8
modules/markup/html_internal_test.go
View file

@ -197,13 +197,13 @@ func TestRender_AutoLink(t *testing.T) {
// render valid commit URLs // render valid commit URLs
tmp := util.URLJoin(AppSubURL, "commit", "d8a994ef243349f321568f9e36d5c3f444b99cae") tmp := util.URLJoin(AppSubURL, "commit", "d8a994ef243349f321568f9e36d5c3f444b99cae")
test(tmp, "<a href=\""+tmp+"\"><code>d8a994ef24</code></a>") test(tmp, "<a href=\""+tmp+"\"><code class=\"nohighlight\">d8a994ef24</code></a>")
tmp += "#diff-2" tmp += "#diff-2"
test(tmp, "<a href=\""+tmp+"\"><code>d8a994ef24 (diff-2)</code></a>") test(tmp, "<a href=\""+tmp+"\"><code class=\"nohighlight\">d8a994ef24 (diff-2)</code></a>")
// render other commit URLs // render other commit URLs
tmp = "https://external-link.gogs.io/gogs/gogs/commit/d8a994ef243349f321568f9e36d5c3f444b99cae#diff-2" tmp = "https://external-link.gitea.io/go-gitea/gitea/commit/d8a994ef243349f321568f9e36d5c3f444b99cae#diff-2"
test(tmp, "<a href=\""+tmp+"\"><code>d8a994ef24 (diff-2)</code></a>") test(tmp, "<a href=\""+tmp+"\"><code class=\"nohighlight\">d8a994ef24 (diff-2)</code></a>")
} }
func TestRender_FullIssueURLs(t *testing.T) { func TestRender_FullIssueURLs(t *testing.T) {