fix(excludeTrailingPunctuationFromURL): add comma to punctuation list

Closes #354
showdownjs · Mar 9, 2017 · fa35fd5 · fa35fd5
1 parent 759a334
commit fa35fd5
Show file tree

Hide file tree

Showing 7 changed files with 14 additions and 8 deletions.
diff --git a/dist/showdown.js b/dist/showdown.js
diff --git a/dist/showdown.js.map b/dist/showdown.js.map
diff --git a/dist/showdown.min.js b/dist/showdown.min.js
diff --git a/dist/showdown.min.js.map b/dist/showdown.min.js.map
diff --git a/src/subParsers/autoLinks.js b/src/subParsers/autoLinks.js
@@ -1,7 +1,7 @@
 // url allowed chars [a-z\d_.~:/?#[]@!$&'()*+,;=-]
 
 var simpleURLRegex  = /\b(((https?|ftp|dict):\/\/|www\.)[^'">\s]+\.[^'">\s]+)()(?=\s|$)(?!["<>])/gi,
-    simpleURLRegex2 = /\b(((https?|ftp|dict):\/\/|www\.)[^'">\s]+\.[^'">\s]+?)([.!?()]?)(?=\s|$)(?!["<>])/gi,
+    simpleURLRegex2 = /\b(((https?|ftp|dict):\/\/|www\.)[^'">\s]+\.[^'">\s]+?)([.!?,()"\[\]]?)(?=\s|$)(?!["<>])/gi,
     //simpleURLRegex3 = /\b(((https?|ftp):\/\/|www\.)[a-z\d.-]+\.[a-z\d_.~:/?#\[\]@!$&'()*+,;=-]+?)([.!?()]?)(?=\s|$)(?!["<>])/gi,
     delimUrlRegex   = /<(((https?|ftp|dict):\/\/|www\.)[^'">\s]+)>/gi,
     simpleMailRegex = /(^|\s)(?:mailto:)?([A-Za-z0-9!#$%&'*+-/=?^_`{|}~.]+@[-a-z0-9]+(\.[-a-z0-9]+)*\.[a-z]+)(?=$|\s)/gmi,

diff --git a/test/features/excludeTrailingPunctuationFromURLs-option.html b/test/features/excludeTrailingPunctuationFromURLs-option.html
@@ -2,3 +2,5 @@
 <p>url <a href="http://www.google.com">http://www.google.com</a>!</p>
 <p>url <a href="http://www.google.com">http://www.google.com</a>? foo</p>
 <p>url (<a href="http://www.google.com">http://www.google.com</a>) bazinga</p>
+<p>url [<a href="http://www.google.com">http://www.google.com</a>] bazinga</p>
+<p>url <a href="http://www.google.com">http://www.google.com</a>, bar</p>
diff --git a/test/features/excludeTrailingPunctuationFromURLs-option.md b/test/features/excludeTrailingPunctuationFromURLs-option.md
@@ -5,3 +5,7 @@ url http://www.google.com!
 url http://www.google.com? foo
 
 url (http://www.google.com) bazinga
+
+url [http://www.google.com] bazinga
+
+url http://www.google.com, bar