Prevent double clicks

This commit is contained in:
Lee Robinson 2023-10-01 12:14:34 -05:00
parent d255138f02
commit 9ef95719a4
4 changed files with 58 additions and 13 deletions

View File

@ -5,7 +5,7 @@ import { addToCart, createCart, getCart, removeFromCart, updateCart } from 'lib/
import { revalidateTag } from 'next/cache'; import { revalidateTag } from 'next/cache';
import { cookies } from 'next/headers'; import { cookies } from 'next/headers';
export async function addItem(prevState: any, selectedVariantId: string) { export async function addItem(prevState: any, selectedVariantId: string | undefined) {
let cartId = cookies().get('cartId')?.value; let cartId = cookies().get('cartId')?.value;
let cart; let cart;

View File

@ -6,6 +6,7 @@ import { addItem } from 'components/cart/actions';
import LoadingDots from 'components/loading-dots'; import LoadingDots from 'components/loading-dots';
import { ProductVariant } from 'lib/shopify/types'; import { ProductVariant } from 'lib/shopify/types';
import { useSearchParams } from 'next/navigation'; import { useSearchParams } from 'next/navigation';
import { useState } from 'react';
import { import {
// @ts-ignore // @ts-ignore
experimental_useFormState as useFormState, experimental_useFormState as useFormState,
@ -14,12 +15,15 @@ import {
function SubmitButton({ function SubmitButton({
availableForSale, availableForSale,
selectedVariantId selectedVariantId,
submitting
}: { }: {
availableForSale: boolean; availableForSale: boolean;
selectedVariantId: string | undefined; selectedVariantId: string | undefined;
submitting: boolean;
}) { }) {
const { pending } = useFormStatus(); let { pending } = useFormStatus();
pending = pending || submitting;
const buttonClasses = const buttonClasses =
'relative flex w-full items-center justify-center rounded-full bg-blue-600 p-4 tracking-wide text-white'; 'relative flex w-full items-center justify-center rounded-full bg-blue-600 p-4 tracking-wide text-white';
const disabledClasses = 'cursor-not-allowed opacity-60 hover:opacity-60'; const disabledClasses = 'cursor-not-allowed opacity-60 hover:opacity-60';
@ -72,6 +76,7 @@ export function AddToCart({
variants: ProductVariant[]; variants: ProductVariant[];
availableForSale: boolean; availableForSale: boolean;
}) { }) {
const [submitting, setSubmitting] = useState(false);
const [message, formAction] = useFormState(addItem, null); const [message, formAction] = useFormState(addItem, null);
const searchParams = useSearchParams(); const searchParams = useSearchParams();
const defaultVariantId = variants.length === 1 ? variants[0]?.id : undefined; const defaultVariantId = variants.length === 1 ? variants[0]?.id : undefined;
@ -84,8 +89,22 @@ export function AddToCart({
const actionWithVariant = formAction.bind(null, selectedVariantId); const actionWithVariant = formAction.bind(null, selectedVariantId);
return ( return (
<form action={actionWithVariant}> <form
<SubmitButton availableForSale={availableForSale} selectedVariantId={selectedVariantId} /> action={actionWithVariant}
// Prevent double clicks
onClick={async (e) => {
e.preventDefault();
if (submitting) return;
setSubmitting(true);
await addItem(message, selectedVariantId);
setSubmitting(false);
}}
>
<SubmitButton
availableForSale={availableForSale}
selectedVariantId={selectedVariantId}
submitting={submitting}
/>
<p aria-live="polite" className="sr-only" role="status"> <p aria-live="polite" className="sr-only" role="status">
{message} {message}
</p> </p>

View File

@ -5,14 +5,16 @@ import clsx from 'clsx';
import { removeItem } from 'components/cart/actions'; import { removeItem } from 'components/cart/actions';
import LoadingDots from 'components/loading-dots'; import LoadingDots from 'components/loading-dots';
import type { CartItem } from 'lib/shopify/types'; import type { CartItem } from 'lib/shopify/types';
import { useState } from 'react';
import { import {
// @ts-ignore // @ts-ignore
experimental_useFormState as useFormState, experimental_useFormState as useFormState,
experimental_useFormStatus as useFormStatus experimental_useFormStatus as useFormStatus
} from 'react-dom'; } from 'react-dom';
function SubmitButton() { function SubmitButton({ submitting }: { submitting: boolean }) {
const { pending } = useFormStatus(); let { pending } = useFormStatus();
pending = pending || submitting;
return ( return (
<button <button
@ -36,13 +38,24 @@ function SubmitButton() {
} }
export function DeleteItemButton({ item }: { item: CartItem }) { export function DeleteItemButton({ item }: { item: CartItem }) {
const [submitting, setSubmitting] = useState(false);
const [message, formAction] = useFormState(removeItem, null); const [message, formAction] = useFormState(removeItem, null);
const itemId = item.id; const itemId = item.id;
const actionWithVariant = formAction.bind(null, itemId); const actionWithVariant = formAction.bind(null, itemId);
return ( return (
<form action={actionWithVariant}> <form
<SubmitButton /> action={actionWithVariant}
// Prevent double clicks
onClick={async (e) => {
e.preventDefault();
if (submitting) return;
setSubmitting(true);
await removeItem(message, itemId);
setSubmitting(false);
}}
>
<SubmitButton submitting={submitting} />
<p aria-live="polite" className="sr-only" role="status"> <p aria-live="polite" className="sr-only" role="status">
{message} {message}
</p> </p>

View File

@ -5,14 +5,16 @@ import clsx from 'clsx';
import { updateItemQuantity } from 'components/cart/actions'; import { updateItemQuantity } from 'components/cart/actions';
import LoadingDots from 'components/loading-dots'; import LoadingDots from 'components/loading-dots';
import type { CartItem } from 'lib/shopify/types'; import type { CartItem } from 'lib/shopify/types';
import { useState } from 'react';
import { import {
// @ts-ignore // @ts-ignore
experimental_useFormState as useFormState, experimental_useFormState as useFormState,
experimental_useFormStatus as useFormStatus experimental_useFormStatus as useFormStatus
} from 'react-dom'; } from 'react-dom';
function SubmitButton({ type }: { type: 'plus' | 'minus' }) { function SubmitButton({ type, submitting }: { type: 'plus' | 'minus'; submitting: boolean }) {
const { pending } = useFormStatus(); let { pending } = useFormStatus();
pending = pending || submitting;
return ( return (
<button <button
@ -39,6 +41,7 @@ function SubmitButton({ type }: { type: 'plus' | 'minus' }) {
} }
export function EditItemQuantityButton({ item, type }: { item: CartItem; type: 'plus' | 'minus' }) { export function EditItemQuantityButton({ item, type }: { item: CartItem; type: 'plus' | 'minus' }) {
const [submitting, setSubmitting] = useState(false);
const [message, formAction] = useFormState(updateItemQuantity, null); const [message, formAction] = useFormState(updateItemQuantity, null);
const payload = { const payload = {
lineId: item.id, lineId: item.id,
@ -48,8 +51,18 @@ export function EditItemQuantityButton({ item, type }: { item: CartItem; type: '
const actionWithVariant = formAction.bind(null, payload); const actionWithVariant = formAction.bind(null, payload);
return ( return (
<form action={actionWithVariant}> <form
<SubmitButton type={type} /> action={actionWithVariant}
// Prevent double clicks
onClick={async (e) => {
e.preventDefault();
if (submitting) return;
setSubmitting(true);
await updateItemQuantity(message, payload);
setSubmitting(false);
}}
>
<SubmitButton type={type} submitting={submitting} />
<p aria-live="polite" className="sr-only" role="status"> <p aria-live="polite" className="sr-only" role="status">
{message} {message}
</p> </p>