create-brainerce-store 1.72.0 → 1.73.0

package/templates/nextjs/ui-canvas/cart/reservation-countdown.tsx

@@ -1,124 +1,124 @@
-'use client';
-
-import { useState, useEffect, useCallback, useRef } from 'react';
-import type { ReservationInfo } from 'brainerce';
-import { useTranslations } from '@/core/lib/translations';
-import { cn } from '@/core/lib/utils';
-
-interface ReservationCountdownProps {
-  reservation: ReservationInfo;
-  /**
-   * Fired once per reservation window when the timer reaches zero, including
-   * when the page opens on an already-expired reservation. Wired to
-   * `useCartPage().onReservationExpired`, which refreshes the cart and blocks
-   * checkout. Keep the prop when you redesign this: the gate lives in the
-   * hook, but nothing tells it about the expiry unless this still calls it.
-   */
-  onExpire?: () => void;
-  className?: string;
-}
-
-/**
- * DESIGN ME — stock-reservation countdown on the cart/checkout pages; the
- * `reservation` prop comes from useCartPage().cart.reservation. Keep the
- * ticking logic, the `onExpire` call and the expired/urgent states (exposed
- * via data-state).
- *
- * Building blocks: shadcn/ui primitives in src/components/ui (Button, Card, Badge, Input, Select, Accordion, Dialog, Skeleton, ...) + lucide-react icons are installed and ready to compose with.
- */
-export function ReservationCountdown({
-  reservation,
-  onExpire,
-  className,
-}: ReservationCountdownProps) {
-  const t = useTranslations('reservation');
-  // `null` means "not measured yet". Server-rendered HTML must not compute a
-  // time, or it disagrees with the client on hydration; starting at 0 instead
-  // would render the expired state for one frame on a perfectly live
-  // reservation, and would fire onExpire on every mount.
-  const [remainingSeconds, setRemainingSeconds] = useState<number | null>(null);
-
-  const calculateRemaining = useCallback(() => {
-    if (!reservation.expiresAt) return 0;
-    const expiresAtMs = new Date(reservation.expiresAt).getTime();
-    const nowMs = Date.now();
-    return Math.max(0, Math.floor((expiresAtMs - nowMs) / 1000));
-  }, [reservation.expiresAt]);
-
-  // Report each expiry window at most once. Keyed on `expiresAt` rather than a
-  // boolean, so the cart refresh that expiry triggers (which remounts this
-  // component) cannot bounce straight back into a second report.
-  const expiresAt = reservation.expiresAt;
-  const reportedExpiryRef = useRef<string | null>(null);
-  const onExpireRef = useRef(onExpire);
-  // Kept in a ref, and synced in an effect rather than during render, so a
-  // caller passing an inline arrow does not restart the timer every render.
-  useEffect(() => {
-    onExpireRef.current = onExpire;
-  }, [onExpire]);
-
-  const reportExpired = useCallback(() => {
-    if (!expiresAt) return;
-    if (reportedExpiryRef.current === expiresAt) return;
-    reportedExpiryRef.current = expiresAt;
-    onExpireRef.current?.();
-  }, [expiresAt]);
-
-  useEffect(() => {
-    const initial = calculateRemaining();
-    setRemainingSeconds(initial);
-    if (initial <= 0) reportExpired();
-
-    const interval = setInterval(() => {
-      const remaining = calculateRemaining();
-      setRemainingSeconds(remaining);
-
-      if (remaining <= 0) {
-        clearInterval(interval);
-        reportExpired();
-      }
-    }, 1000);
-
-    return () => clearInterval(interval);
-  }, [calculateRemaining, reportExpired]);
-
-  if (!reservation.hasReservation) return null;
-
-  const displaySeconds = remainingSeconds ?? 0;
-  const minutes = Math.floor(displaySeconds / 60);
-  const seconds = displaySeconds % 60;
-  const isExpired = remainingSeconds !== null && remainingSeconds <= 0;
-  const isUrgent = displaySeconds > 0 && displaySeconds < 120;
-
-  const displayMessage = reservation.countdownMessage
-    ? reservation.countdownMessage.replace(
-        '{time}',
-        `${minutes}:${seconds.toString().padStart(2, '0')}`
-      )
-    : null;
-
-  return (
-    <div
-      role="timer"
-      data-state={isExpired ? 'expired' : isUrgent ? 'urgent' : 'active'}
-      className={cn(className)}
-    >
-      {isExpired ? (
-        <>
-          <p>{t('expired')}</p>
-          <p>{t('expiredHint')}</p>
-        </>
-      ) : displayMessage ? (
-        <p>{displayMessage}</p>
-      ) : (
-        <p>
-          {isUrgent ? `${t('hurry')} ` : ''}
-          {t('reservedFor')}{' '}
-          <span>
-            {minutes}:{seconds.toString().padStart(2, '0')}
-          </span>
-        </p>
-      )}
-    </div>
-  );
-}
+'use client';
+
+import { useState, useEffect, useCallback, useRef } from 'react';
+import type { ReservationInfo } from 'brainerce';
+import { useTranslations } from '@/core/lib/translations';
+import { cn } from '@/core/lib/utils';
+
+interface ReservationCountdownProps {
+  reservation: ReservationInfo;
+  /**
+   * Fired once per reservation window when the timer reaches zero, including
+   * when the page opens on an already-expired reservation. Wired to
+   * `useCartPage().onReservationExpired`, which refreshes the cart and blocks
+   * checkout. Keep the prop when you redesign this: the gate lives in the
+   * hook, but nothing tells it about the expiry unless this still calls it.
+   */
+  onExpire?: () => void;
+  className?: string;
+}
+
+/**
+ * DESIGN ME — stock-reservation countdown on the cart/checkout pages; the
+ * `reservation` prop comes from useCartPage().cart.reservation. Keep the
+ * ticking logic, the `onExpire` call and the expired/urgent states (exposed
+ * via data-state).
+ *
+ * Building blocks: shadcn/ui primitives in src/components/ui (Button, Card, Badge, Input, Select, Accordion, Dialog, Skeleton, ...) + lucide-react icons are installed and ready to compose with.
+ */
+export function ReservationCountdown({
+  reservation,
+  onExpire,
+  className,
+}: ReservationCountdownProps) {
+  const t = useTranslations('reservation');
+  // `null` means "not measured yet". Server-rendered HTML must not compute a
+  // time, or it disagrees with the client on hydration; starting at 0 instead
+  // would render the expired state for one frame on a perfectly live
+  // reservation, and would fire onExpire on every mount.
+  const [remainingSeconds, setRemainingSeconds] = useState<number | null>(null);
+
+  const calculateRemaining = useCallback(() => {
+    if (!reservation.expiresAt) return 0;
+    const expiresAtMs = new Date(reservation.expiresAt).getTime();
+    const nowMs = Date.now();
+    return Math.max(0, Math.floor((expiresAtMs - nowMs) / 1000));
+  }, [reservation.expiresAt]);
+
+  // Report each expiry window at most once. Keyed on `expiresAt` rather than a
+  // boolean, so the cart refresh that expiry triggers (which remounts this
+  // component) cannot bounce straight back into a second report.
+  const expiresAt = reservation.expiresAt;
+  const reportedExpiryRef = useRef<string | null>(null);
+  const onExpireRef = useRef(onExpire);
+  // Kept in a ref, and synced in an effect rather than during render, so a
+  // caller passing an inline arrow does not restart the timer every render.
+  useEffect(() => {
+    onExpireRef.current = onExpire;
+  }, [onExpire]);
+
+  const reportExpired = useCallback(() => {
+    if (!expiresAt) return;
+    if (reportedExpiryRef.current === expiresAt) return;
+    reportedExpiryRef.current = expiresAt;
+    onExpireRef.current?.();
+  }, [expiresAt]);
+
+  useEffect(() => {
+    const initial = calculateRemaining();
+    setRemainingSeconds(initial);
+    if (initial <= 0) reportExpired();
+
+    const interval = setInterval(() => {
+      const remaining = calculateRemaining();
+      setRemainingSeconds(remaining);
+
+      if (remaining <= 0) {
+        clearInterval(interval);
+        reportExpired();
+      }
+    }, 1000);
+
+    return () => clearInterval(interval);
+  }, [calculateRemaining, reportExpired]);
+
+  if (!reservation.hasReservation) return null;
+
+  const displaySeconds = remainingSeconds ?? 0;
+  const minutes = Math.floor(displaySeconds / 60);
+  const seconds = displaySeconds % 60;
+  const isExpired = remainingSeconds !== null && remainingSeconds <= 0;
+  const isUrgent = displaySeconds > 0 && displaySeconds < 120;
+
+  const displayMessage = reservation.countdownMessage
+    ? reservation.countdownMessage.replace(
+        '{time}',
+        `${minutes}:${seconds.toString().padStart(2, '0')}`
+      )
+    : null;
+
+  return (
+    <div
+      role="timer"
+      data-state={isExpired ? 'expired' : isUrgent ? 'urgent' : 'active'}
+      className={cn(className)}
+    >
+      {isExpired ? (
+        <>
+          <p>{t('expired')}</p>
+          <p>{t('expiredHint')}</p>
+        </>
+      ) : displayMessage ? (
+        <p>{displayMessage}</p>
+      ) : (
+        <p>
+          {isUrgent ? `${t('hurry')} ` : ''}
+          {t('reservedFor')}{' '}
+          <span>
+            {minutes}:{seconds.toString().padStart(2, '0')}
+          </span>
+        </p>
+      )}
+    </div>
+  );
+}

package/templates/nextjs/ui-canvas/product/review-form.tsx

@@ -323,7 +323,15 @@ function ReviewEditor({
       </label>
 
       {/* DESIGN ME — review photo picker. Renders only when the store allows
-          photos; every limit comes from `photos`, never hard-coded. */}
+          photos; every limit comes from `photos`, never hard-coded.
+          Two things to get right when you style it. Wrap the input in a <label>
+          and hide the input with sr-only: the browser's own "Choose Files" button
+          cannot be styled and will look nothing like the rest of your form. Then
+          paint your own disabled state, because `disabled` on a hidden input greys
+          NOTHING — at photos.maxPerReview the control would look alive and do
+          nothing. Say why it is disabled. Keep it a real input (no ref.click()) so
+          keyboard and assistive tech reach it, and add focus-within so the focus
+          ring the native control gave you for free survives. */}
       {photos.enabled && (
         <fieldset>
           <legend>