##// 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 result += " title=\"" + title + "\"";
590 result += " title=\"" + title + "\"";
591 }
591 }
592
592
593 result += ">" + link_text + "</a>";
593 result += " target=\"_blank\">" + link_text + "</a>";
594
594
595 return result;
595 return result;
596 }
596 }
@@ -1183,7 +1183,8 else
1183
1183
1184 function _DoAutoLinks(text) {
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 // *except* for the <http://www.foo.com> case
1188 // *except* for the <http://www.foo.com> case
1188
1189
1189 // automatically add < and > around unadorned raw hyperlinks
1190 // automatically add < and > around unadorned raw hyperlinks
@@ -1192,7 +1193,7 else
1192
1193
1193 // autolink anything like <http://example.com>
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 text = text.replace(/<((https?|ftp):[^'">\s]+)>/gi, replacer);
1197 text = text.replace(/<((https?|ftp):[^'">\s]+)>/gi, replacer);
1197
1198
1198 // Email addresses: <address@domain.foo>
1199 // Email addresses: <address@domain.foo>
General Comments 0
You need to be logged in to leave comments. Login now