fix link title w. quote, multiple headings in link

JohannesKaufmann · JohannesKaufmann · commit 9cd4bbdb0361 · 2021-06-13T20:50:30.000+02:00
diff --git a/commonmark.go b/commonmark.go
@@ -115,6 +115,7 @@ var commonmark = []Rule{
 			insideLink := selec.ParentsFiltered("a").Length() > 0
 			if insideLink {
 				text := opt.StrongDelimiter + content + opt.StrongDelimiter
+				text = AddSpaceIfNessesary(selec, text)
 				return &text
 			}
 
@@ -219,6 +220,8 @@ var commonmark = []Rule{
 			var title string
 			if t, ok := selec.Attr("title"); ok {
 				t = strings.Replace(t, "\n", " ", -1)
+				// escape all quotes
+				t = strings.Replace(t, `"`, `\"`, -1)
 				title = fmt.Sprintf(` "%s"`, t)
 			}
 
diff --git a/testdata/TestCommonmark/link/input.html b/testdata/TestCommonmark/link/input.html
@@ -82,3 +82,22 @@
 <!--links directly next to each other-->
 <p>Before<a href="http://example_close.com">close</a>After</p>
 <br />
+
+
+<!--link with headings-->
+<a href="/page.html">
+	<h4>Heading A</h4>
+	<h3>Heading B</h3>
+</a>
+<br />
+
+
+<!--link with quotes in title-->
+<a
+	href="/page.html"
+	title="&quot;Vermögenssteuer ist aus wirtschaftlicher Sicht klug&quot;"
+	>
+	<span class="teaser__kicker">DIW-Chef zum Grünen-Programm</span>
+	<span class="teaser__headline"> "Vermögenssteuer ist aus wirtschaftlicher Sicht klug"</span>
+</a>
+<br />
diff --git a/testdata/TestCommonmark/link/output.inlined.golden b/testdata/TestCommonmark/link/output.inlined.golden
@@ -25,4 +25,8 @@ Second Text](http://multi.org/)
 
 [first left](http://first_next.com) [second right](http://second_next.com)
 
-Before [close](http://example_close.com) After
+Before [close](http://example_close.com) After
+
+[**Heading A**  **Heading B**](http://example.com/page.html)
+
+[DIW-Chef zum Grünen-Programm "Vermögenssteuer ist aus wirtschaftlicher Sicht klug"](http://example.com/page.html "\"Vermögenssteuer ist aus wirtschaftlicher Sicht klug\"")
diff --git a/testdata/TestCommonmark/link/output.referenced_collapsed.golden b/testdata/TestCommonmark/link/output.referenced_collapsed.golden
@@ -27,6 +27,10 @@ Second Text][]
 
 Before [close][] After
 
+[**Heading A**  **Heading B**][]
+
+[DIW-Chef zum Grünen-Programm "Vermögenssteuer ist aus wirtschaftlicher Sicht klug"][]
+
 [Simple Absolute Link]: http://simple.org/
 [Simple Relative Link]: http://example.com/page.html
 [Link with Space]: http://space.org/
@@ -45,4 +49,6 @@ Second Text]: http://list.org/
 [second below]: http://second_under.com
 [first left]: http://first_next.com
 [second right]: http://second_next.com
-[close]: http://example_close.com
+[close]: http://example_close.com
+[**Heading A**  **Heading B**]: http://example.com/page.html
+[DIW-Chef zum Grünen-Programm "Vermögenssteuer ist aus wirtschaftlicher Sicht klug"]: http://example.com/page.html "\"Vermögenssteuer ist aus wirtschaftlicher Sicht klug\""
diff --git a/testdata/TestCommonmark/link/output.referenced_full.golden b/testdata/TestCommonmark/link/output.referenced_full.golden
@@ -27,6 +27,10 @@ Second Text][7]
 
 Before [close][14] After
 
+[**Heading A**  **Heading B**][15]
+
+[DIW-Chef zum Grünen-Programm "Vermögenssteuer ist aus wirtschaftlicher Sicht klug"][16]
+
 [1]: http://simple.org/
 [2]: http://example.com/page.html
 [3]: http://space.org/
@@ -40,4 +44,6 @@ Before [close][14] After
 [11]: http://second_under.com
 [12]: http://first_next.com
 [13]: http://second_next.com
-[14]: http://example_close.com
+[14]: http://example_close.com
+[15]: http://example.com/page.html
+[16]: http://example.com/page.html "\"Vermögenssteuer ist aus wirtschaftlicher Sicht klug\""
diff --git a/testdata/TestCommonmark/link/output.referenced_shortcut.golden b/testdata/TestCommonmark/link/output.referenced_shortcut.golden
@@ -27,6 +27,10 @@ Second Text]
 
 Before [close] After
 
+[**Heading A**  **Heading B**]
+
+[DIW-Chef zum Grünen-Programm "Vermögenssteuer ist aus wirtschaftlicher Sicht klug"]
+
 [Simple Absolute Link]: http://simple.org/
 [Simple Relative Link]: http://example.com/page.html
 [Link with Space]: http://space.org/
@@ -45,4 +49,6 @@ Second Text]: http://list.org/
 [second below]: http://second_under.com
 [first left]: http://first_next.com
 [second right]: http://second_next.com
-[close]: http://example_close.com
+[close]: http://example_close.com
+[**Heading A**  **Heading B**]: http://example.com/page.html
+[DIW-Chef zum Grünen-Programm "Vermögenssteuer ist aus wirtschaftlicher Sicht klug"]: http://example.com/page.html "\"Vermögenssteuer ist aus wirtschaftlicher Sicht klug\""
diff --git a/testdata/TestCommonmark/link/output.relative.golden b/testdata/TestCommonmark/link/output.relative.golden
@@ -25,4 +25,8 @@ Second Text](http://multi.org/)
 
 [first left](http://first_next.com) [second right](http://second_next.com)
 
-Before [close](http://example_close.com) After
+Before [close](http://example_close.com) After
+
+[**Heading A**  **Heading B**](/page.html)
+
+[DIW-Chef zum Grünen-Programm "Vermögenssteuer ist aus wirtschaftlicher Sicht klug"](/page.html "\"Vermögenssteuer ist aus wirtschaftlicher Sicht klug\"")

Original file line number	Diff line number	Diff line change
`@@ -115,6 +115,7 @@ var commonmark = []Rule{`
`115`	`115`	`insideLink := selec.ParentsFiltered("a").Length() > 0`
`116`	`116`	`if insideLink {`
`117`	`117`	`text := opt.StrongDelimiter + content + opt.StrongDelimiter`
	`118`	`+ text = AddSpaceIfNessesary(selec, text)`
`118`	`119`	`return &text`
`119`	`120`	`}`
`120`	`121`
`@@ -219,6 +220,8 @@ var commonmark = []Rule{`
`219`	`220`	`var title string`
`220`	`221`	`if t, ok := selec.Attr("title"); ok {`
`221`	`222`	`t = strings.Replace(t, "\n", " ", -1)`
	`223`	`+ // escape all quotes`
	`224`	+ t = strings.Replace(t, `"`, `\"`, -1)
`222`	`225`	title = fmt.Sprintf(` "%s"`, t)
`223`	`226`	`}`
`224`	`227`