Looking for some insight into amending a single line of code

[whatzmaddy]

Hey @yodog.. I am using your super helpful script stackoverflow-code-box-options.user.js & looking for some help to amend one thing for personal use.

Within the function part fnReplaceLinks(), you have this code line return this.href.match(/^http/) && this.hostname !== location.hostname; I would like to change it into something in the lines of "While I am on stackoverflow, do not open stackoverflow links in a new tab unless if the link is explicitly https://stackoverflow.com/q*

Any insight on how to amend this would be appreciated:)

[yodog]

the ideia behind fnReplaceLinks() is to open all externals links in a new tab/window instead of taking the user away from the current page.

it is not supposed to open internal links (from the same domain, like "stackoverflow.com" in your browser address bar) in new tabs, and if it is, you found a bug :)

function fnReplaceLinks() {
    [...document.links].forEach(
        a => $(a).filter(function() {
            return this.href.match(/^http/) && this.hostname !== location.hostname;
        }).attr('target', '_blank')
    );
}

as you already know, you have the change the rule to open links in new tabs in the line return this.href.match(/^http/) && this.hostname !== location.hostname;

if you want to keep the current rules and only add yours, you can change match to something like this.href.match(/^http.*\/q/) or a more refined regex. feel fre to experiment with it.

[whatzmaddy]

Thanks for your prompt response. Arrived at the following working code with some help, which works for my use-case

return this.hostname!== location.hostname || (this.hostname === location.hostname && (this.href.includes('/questions/') || this.href.includes('/q/')))