Merge pull request #941 from qtchaos/fix-download

Fix media download button redirecting to incorrect URL on main tab
This commit is contained in:
William Oldham 2024-02-26 15:09:35 +00:00 committed by GitHub
commit 02fceb7f8e
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -1,6 +1,5 @@
import classNames from "classnames";
import { ReactNode, useCallback } from "react";
import { useNavigate } from "react-router-dom";
import { Icon, Icons } from "@/components/Icon";
import { Spinner } from "@/components/layout/Spinner";
@ -21,7 +20,6 @@ interface Props {
}
export function Button(props: Props) {
const navigate = useNavigate();
const { onClick, href, loading } = props;
const cb = useCallback(
(
@ -31,10 +29,12 @@ export function Button(props: Props) {
>,
) => {
if (loading) return;
if (href && !onClick) navigate(href);
else onClick?.(event);
if (href && !onClick) {
event.preventDefault();
window.open(href, "_blank", "noreferrer");
} else onClick?.(event);
},
[onClick, href, navigate, loading],
[onClick, href, loading],
);
let colorClasses = "bg-white hover:bg-gray-200 text-black";