Skip to content

Commit

Permalink
Merge pull request #21 from risen228/fix/allow-link-modified-events
Browse files Browse the repository at this point in the history
[Fix] Skip navigation for modified keys events and some "target" values
  • Loading branch information
Kelin2025 authored Jul 31, 2023
2 parents e67ce85 + 101bae0 commit 84dd970
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions src/link.tsx
Original file line number Diff line number Diff line change
@@ -47,6 +47,7 @@ const RouteLinkView = <Params extends RouteParams>(
inactiveClassName,
onClick,
children,
target,
...linkProps
} = props;

@@ -72,6 +73,7 @@ const RouteLinkView = <Params extends RouteParams>(
href={href}
{...linkProps}
className={clsx(className, isOpened ? activeClassName : inactiveClassName)}
target={target}
onClick={(evt) => {
if (onClick) {
onClick(evt);
@@ -82,6 +84,16 @@ const RouteLinkView = <Params extends RouteParams>(
return
}

// let browser handle "_blank" target and etc
if (target && target !== '_self') {
return
}

// skip modified events (like cmd + click to open the link in new tab)
if (evt.metaKey || evt.altKey || evt.ctrlKey || evt.shiftKey) {
return
}

evt.preventDefault();
navigate({
params: params || ({} as Params),

0 comments on commit 84dd970

Please sign in to comment.