##// END OF EJS Templates
Added target="_blank" to hyperlinks in Markdown.Converter.js so that clicking on a link opens it in a new window. Fixes Issue #1701 and adds surrounding quotes.
Roy Hyunjin Han -
Show More
@@ -590,7 +590,7 else
590 590 result += " title=\"" + title + "\"";
591 591 }
592 592
593 result += ">" + link_text + "</a>";
593 result += " target=\"_blank\">" + link_text + "</a>";
594 594
595 595 return result;
596 596 }
@@ -1183,7 +1183,8 else
1183 1183
1184 1184 function _DoAutoLinks(text) {
1185 1185
1186 // note that at this point, all other URL in the text are already hyperlinked as <a href=""></a>
1186 // note that at this point, all other URL in the text are already
1187 // hyperlinked as <a href="" target="_blank"></a>
1187 1188 // *except* for the <http://www.foo.com> case
1188 1189
1189 1190 // automatically add < and > around unadorned raw hyperlinks
@@ -1192,7 +1193,7 else
1192 1193
1193 1194 // autolink anything like <http://example.com>
1194 1195
1195 var replacer = function (wholematch, m1) { return "<a href=\"" + m1 + "\">" + pluginHooks.plainLinkText(m1) + "</a>"; }
1196 var replacer = function (wholematch, m1) { return "<a href=\"" + m1 + "\" target=\"_blank\"">" + pluginHooks.plainLinkText(m1) + "</a>"; }
1196 1197 text = text.replace(/<((https?|ftp):[^'">\s]+)>/gi, replacer);
1197 1198
1198 1199 // Email addresses: <address@domain.foo>
General Comments 0
You need to be logged in to leave comments. Login now