Browse Source

Proper onClick types and give <a> the callback

pull/860/head
Astrid 2 years ago
parent
commit
a562cbeb25
  1. 21
      src/components/buttons/Button.tsx

21
src/components/buttons/Button.tsx

@ -7,7 +7,9 @@ import { Spinner } from "@/components/layout/Spinner"; @@ -7,7 +7,9 @@ import { Spinner } from "@/components/layout/Spinner";
interface Props {
icon?: Icons;
onClick?: () => void;
onClick?: (
event: React.MouseEvent<HTMLAnchorElement | HTMLButtonElement, MouseEvent>,
) => void;
children?: ReactNode;
theme?: "white" | "purple" | "secondary" | "danger";
padding?: string;
@ -21,11 +23,19 @@ interface Props { @@ -21,11 +23,19 @@ interface Props {
export function Button(props: Props) {
const navigate = useNavigate();
const { onClick, href, loading } = props;
const cb = useCallback(() => {
const cb = useCallback(
(
event: React.MouseEvent<
HTMLAnchorElement | HTMLButtonElement,
MouseEvent
>,
) => {
if (loading) return;
if (href) navigate(href);
else onClick?.();
}, [onClick, href, navigate, loading]);
if (href && !onClick) navigate(href);
else onClick?.(event);
},
[onClick, href, navigate, loading],
);
let colorClasses = "bg-white hover:bg-gray-200 text-black";
if (props.theme === "purple")
@ -80,6 +90,7 @@ export function Button(props: Props) { @@ -80,6 +90,7 @@ export function Button(props: Props) {
target="_blank"
rel="noreferrer"
download={props.download}
onClick={cb}
>
{content}
</a>

Loading…
Cancel
Save