feat: [feat/doc-updates] open external links in new tab/window

[ReneHezser]

Overview/Summary

This is a PR not for the main branch

External links will be opened in a new tab instead of navigating away from the current page.

This PR fixes/adds/changes/removes

none

Breaking Changes

none

As part of this Pull Request I have

• Read the Contribution Guide and ensured this PR is compliant with the guide
• Checked for duplicate Pull Requests
• Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
• Ensured my code/branch is up-to-date with the latest changes in the main branch
• Ensured PR tests are passing
• Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

[ReneHezser]

@matebarabas the first PR for the doc update branch

[AlexanderSehr]

Hey @ReneHezser,
would that then be the case for literally all links? I'd usually use my mousewheel if I want to open somthing in a new tab and otherwise would always want to navigate to a given page. Not saying we cannot change it - but if this works like I think it will, then we're essentially limiting the user 😄

Asking for a friend

[ReneHezser]

Asking for a friend

Hey @AlexanderSehr and Alex's friend
I understand the concerns your friend has. The web is divided if it is best to open external links (pointing to another domain) should be opened in a new tab/window, or not. In our case, I would like to do so, as they provide additional information to read, but would navigate away from the "follow me guide" on the site.

[matebarabas]

A few comments/thoughts:

• I think opening external links in a new tab is common best practice, so I think we should adapt this practice - which as per the intent does just that, even if it looks like it would result in false positives. :)
• We need to test/make sure that this solution really only gets triggered in case of external links (i.e., stuff outside of the AVM website, including any of the GH repos, etc.)
• We need to test/make sure that this gets triggered in case of http and https links.
• This PR is not for main, but for a long-standing documentation branch, we'll collaborate on and release to main when we're ready for the first wave of changes.