fix: Check button property and modifier key states before calling `…

…preventDefault()` (#473)

addon/link.ts

@@ -13,6 +13,8 @@ import { tracked } from '@glimmer/tracking';
 
 import LinkManagerService from './services/link-manager';
 
+const MAIN_BUTTON = 0;
+
 export type QueryParams = Record<string, unknown>;
 
 export function isQueryParams(
@@ -54,6 +56,14 @@ function freezeParams(params: LinkParams) {
   return params;
 }
 
+function isUnmodifiedLeftClick(event: MouseEvent): boolean {
+  return event.button === MAIN_BUTTON && !event.ctrlKey && !event.metaKey;
+}
+
+function isMouseEvent(event: unknown): event is MouseEvent {
+  return typeof event === 'object' && event !== null && 'button' in event;
+}
+
 export default class Link {
   @tracked
   // eslint-disable-next-line @typescript-eslint/naming-convention
@@ -262,7 +272,8 @@ export class UILink extends Link {
   private preventDefault(event?: Event | unknown) {
     if (
       (this._params.preventDefault ?? true) &&
-      typeof (event as Event)?.preventDefault === 'function'
+      typeof (event as Event)?.preventDefault === 'function' &&
+      (!isMouseEvent(event) || isUnmodifiedLeftClick(event))
     ) {
       (event as Event).preventDefault();
     }