From: Bjørn Erik Pedersen Date: Sun, 5 Jan 2020 14:58:14 +0000 (+0100) Subject: markup/goldmark: Adjust auto ID space handling X-Git-Tag: v0.62.2~4 X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=9b6e61464b09ffe3423fb8d7c72bddb7a9ed5b98;p=brevno-suite%2Fhugo markup/goldmark: Adjust auto ID space handling GitHub does not consider tabs as delimiter, see https://github.com/bep/portable-hugo-links/blob/master/blog/p2.md Closes #6710 --- diff --git a/markup/goldmark/autoid.go b/markup/goldmark/autoid.go index 950d4a57..72cb79f7 100644 --- a/markup/goldmark/autoid.go +++ b/markup/goldmark/autoid.go @@ -58,7 +58,7 @@ func sanitizeAnchorNameWithHook(b []byte, idType string, hook func(buf *bytes.Bu r, size := utf8.DecodeRune(b) switch { case asciiOnly && size != 1: - case r == '-' || isSpace(r): + case r == '-' || r == ' ': buf.WriteRune('-') case isAlphaNumeric(r): buf.WriteRune(unicode.ToLower(r)) @@ -85,10 +85,6 @@ func isAlphaNumeric(r rune) bool { return r == '_' || unicode.IsLetter(r) || unicode.IsDigit(r) } -func isSpace(r rune) bool { - return r == ' ' || r == '\t' -} - var _ parser.IDs = (*idFactory)(nil) type idFactory struct { diff --git a/markup/goldmark/autoid_test.go b/markup/goldmark/autoid_test.go index 1257b348..0ddf5e88 100644 --- a/markup/goldmark/autoid_test.go +++ b/markup/goldmark/autoid_test.go @@ -40,6 +40,8 @@ Many spaces here Forward/slash Backward\slash Under_score +Nonbreaking Space +Tab Space ` expect := ` @@ -56,6 +58,8 @@ many---spaces--here forwardslash backwardslash under_score +nonbreakingspace +tabspace ` tests, expect = strings.TrimSpace(tests), strings.TrimSpace(expect)