incorporate review

This commit is contained in:
Matthew Hodgson 2016-03-21 15:45:04 +00:00
parent d54a75c913
commit c3e96f8af1
2 changed files with 5 additions and 7 deletions

View file

@ -47,12 +47,12 @@ var sanitizeHtmlParams = {
transformTags: { // custom to matrix transformTags: { // custom to matrix
// add blank targets to all hyperlinks except vector URLs // add blank targets to all hyperlinks except vector URLs
'a': function(tagName, attribs) { 'a': function(tagName, attribs) {
var m = attribs.href.match(linkifyMatrix.VECTOR_URL_PATTERN); var m = attribs.href ? attribs.href.match(linkifyMatrix.VECTOR_URL_PATTERN) : null;
if (m) { if (m) {
return { tagName: 'a', attribs: { href: attribs.href } }; return { tagName: 'a', attribs: { href: attribs.href, name: attribs.name } };
} }
else { else {
return { tagName: 'a', attribs: { href: attribs.href, target: '_blank'} }; return { tagName: 'a', attribs: { href: attribs.href, name: attribs.name, target: '_blank' } };
} }
}, },
}, },

View file

@ -106,7 +106,7 @@ var escapeRegExp = function(string) {
// as if someone explicitly links to vector.im/develop and we're on vector.im/beta // as if someone explicitly links to vector.im/develop and we're on vector.im/beta
// they may well be trying to get us to explicitly go to develop. // they may well be trying to get us to explicitly go to develop.
// FIXME: intercept matrix.to URLs as well. // FIXME: intercept matrix.to URLs as well.
matrixLinkify.VECTOR_URL_PATTERN = "(https?:\/\/)?" + escapeRegExp(window.location.host + window.location.pathname); matrixLinkify.VECTOR_URL_PATTERN = "^(https?:\/\/)?" + escapeRegExp(window.location.host + window.location.pathname);
matrixLinkify.options = { matrixLinkify.options = {
events: function (href, type) { events: function (href, type) {
@ -146,9 +146,7 @@ matrixLinkify.options = {
return '_blank'; return '_blank';
} }
} }
else { return null;
return '_blank';
}
}, },
}; };