@spree/dashboard-core 0.13.1 → 1.0.0-beta.1

package/src/products/variant-edit-sheet.tsx

@@ -0,0 +1,725 @@
+import type { ProductFormValues, VariantFormValues } from '@spree/dashboard-core'
+import { CountryCombobox, StoreDatePicker } from '@spree/dashboard-core'
+import {
+  Alert,
+  AlertDescription,
+  Button,
+  Field,
+  FieldDescription,
+  FieldError,
+  FieldLabel,
+  FormSection,
+  Input,
+  InputGroup,
+  InputGroupAddon,
+  InputGroupInput,
+  InputGroupText,
+  Select,
+  SelectContent,
+  SelectItem,
+  SelectTrigger,
+  SelectValue,
+  Sheet,
+  SheetContent,
+  SheetDescription,
+  SheetFooter,
+  SheetHeader,
+  SheetTitle,
+  Switch,
+} from '@spree/dashboard-ui'
+import { Link } from '@tanstack/react-router'
+import { InfoIcon } from 'lucide-react'
+import { useEffect, useRef } from 'react'
+import { Controller, type UseFormReturn } from 'react-hook-form'
+import { Trans, useTranslation } from 'react-i18next'
+import { useTenantId } from '../providers/tenant-provider'
+import { cartonSize } from './carton-size'
+import { normalizeCustomsDescription, normalizeHsCode } from './normalize-customs'
+import { PURCHASE_UNITS } from './normalize-quantity'
+import {
+  useFormCartonPackageTypes as useCartonPackageTypes,
+  useFormOptionTypes as useOptionTypes,
+  useFormTaxCategories as useTaxCategories,
+} from './use-product-form-data'
+import { variantDisplayLabel } from './variants-matrix'
+
+const WEIGHT_UNITS = ['g', 'kg', 'lb', 'oz'] as const
+const DIMENSION_UNITS = ['mm', 'cm', 'in'] as const
+
+interface Props {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  form: UseFormReturn<ProductFormValues, any, any>
+  variantIndex: number
+  open: boolean
+  onOpenChange: (open: boolean) => void
+}
+
+// Per-variant detail editor. Binds to variants.${i}.* on the parent product
+// form — no own form state. Save closes the sheet (changes are already in
+// form state; the product Save button commits them). Cancel restores the
+// snapshot captured when the sheet opened.
+export function VariantEditSheet({ form, variantIndex, open, onOpenChange }: Props) {
+  const { t } = useTranslation()
+  const { data: taxCategoriesResponse } = useTaxCategories()
+  const taxCategories = taxCategoriesResponse?.data ?? []
+  const hasTaxCategories = taxCategories.length > 0
+  const { data: optionTypesData } = useOptionTypes()
+  const optionTypes = optionTypesData?.data ?? []
+  const tenantId = useTenantId()
+  // A panel that does not manage package types at all (a seller's) has no
+  // packing to ask about, so the section stays hidden there. An operator with
+  // none created yet still sees it, disabled, saying where to make one —
+  // hiding it outright left no trace of a feature they had not set up.
+  const {
+    data: cartonTypesResponse,
+    isPending: cartonTypesPending,
+    supported: cartonTypesSupported,
+  } = useCartonPackageTypes()
+  const cartonTypes = cartonTypesResponse?.data ?? []
+  const hasCartonTypes = cartonTypes.length > 0
+  // Only once the list has actually answered. While it is in flight the empty
+  // array below is not yet evidence of an empty store, and telling a merchant
+  // who has cartons to go and make one is worse than saying nothing.
+  const knownEmpty = cartonTypesSupported && !cartonTypesPending && !hasCartonTypes
+  // A packed carton is weighed on the same scale as the goods inside it, so
+  // this follows the variant's own unit rather than offering a second one.
+  const cartonWeightUnit = form.watch(`variants.${variantIndex}.weight_unit`) || ''
+
+  // Snapshot the variant when the sheet opens so Cancel can restore it.
+  // Re-snapshot if the user switches between variant rows without closing
+  // (variantIndex changes while open) so the next Cancel restores the right row.
+  //
+  // Deep-clone via structuredClone — `form.getValues` returns a reference
+  // to RHF's internal field state, and the inline cell editors mutate
+  // nested arrays (prices, stock_levels) in place. Without the clone, Cancel
+  // would write back the already-edited object.
+  //
+  // We also stash the variant's `id` (for persisted rows). The sheet is
+  // modal so it's hard but not impossible for the matrix array to reorder
+  // underneath (e.g. keyboard reorder while focus is in the sheet). On
+  // Cancel, re-resolve the index by id and only restore if the snapshot
+  // still maps to a unique row — otherwise drop the restore rather than
+  // overwriting a different variant's values.
+  const snapshotRef = useRef<{ value: VariantFormValues; id: string | undefined } | null>(null)
+  useEffect(() => {
+    if (!open) {
+      snapshotRef.current = null
+      return
+    }
+    const current = form.getValues(`variants.${variantIndex}`)
+    snapshotRef.current = current
+      ? { value: structuredClone(current) as VariantFormValues, id: current.id }
+      : null
+  }, [open, variantIndex, form])
+
+  const variant = form.watch(`variants.${variantIndex}`)
+  if (!variant) return null
+
+  const label = variantDisplayLabel(
+    variant,
+    t('admin.products.variants.default_variant'),
+    optionTypes,
+  )
+
+  const handleCancel = () => {
+    const snap = snapshotRef.current
+    if (snap) {
+      // Resolve the target index by stable id when we have one — the array
+      // may have reordered while the sheet was open. For unsaved rows
+      // (no id) trust the current index.
+      let targetIndex = variantIndex
+      if (snap.id) {
+        const all = form.getValues('variants') ?? []
+        const found = all.findIndex((v) => v.id === snap.id)
+        if (found === -1) return onOpenChange(false)
+        targetIndex = found
+      }
+      // Restore the snapshot via resetField so the variant's dirty bit is
+      // cleared too — `setValue` with `shouldDirty: true` would leave the
+      // form falsely dirty after a no-op cancel (same trap MediaEditSheet
+      // hit). `resetField` re-baselines just this subtree, leaving sibling
+      // dirty fields elsewhere on the form untouched.
+      form.resetField(`variants.${targetIndex}`, { defaultValue: snap.value })
+    }
+    onOpenChange(false)
+  }
+
+  const handleDone = () => {
+    onOpenChange(false)
+  }
+
+  return (
+    <Sheet open={open} onOpenChange={(o) => (o ? onOpenChange(o) : handleCancel())}>
+      <SheetContent side="right" className="flex flex-col">
+        <SheetHeader>
+          <SheetTitle>{t('admin.products.variants.edit_variant', { name: label })}</SheetTitle>
+          {/* Nothing here is written until the product itself is saved, and a
+              sheet that closes with a tick invites the opposite reading. */}
+          <SheetDescription>{t('admin.products.variants.sheet.pending_save')}</SheetDescription>
+        </SheetHeader>
+
+        <div className="flex-1 overflow-y-auto p-4 flex flex-col gap-6">
+          <FormSection title={t('admin.products.variants.sheet.identity')}>
+            <Field>
+              <FieldLabel htmlFor={`variant-${variantIndex}-sku`}>
+                {t('admin.fields.variant.sku.label')}
+              </FieldLabel>
+              {/*
+                Use Controller instead of register here: the matrix row in
+                VariantsSection also registers `variants.${i}.sku`, and two
+                `register` calls on the same field path share a single ref
+                slot — the second mount wins, so typing in one input never
+                updates the other's display. A Controller subscribes to the
+                field via useController and renders a controlled input, so
+                both surfaces stay in sync whichever one the merchant edits.
+              */}
+              <Controller
+                name={`variants.${variantIndex}.sku`}
+                control={form.control}
+                render={({ field }) => (
+                  <Input
+                    id={`variant-${variantIndex}-sku`}
+                    placeholder={t('admin.fields.variant.sku.placeholder')}
+                    value={field.value ?? ''}
+                    onChange={(e) => field.onChange(e.target.value)}
+                    onBlur={field.onBlur}
+                  />
+                )}
+              />
+              <FieldError errors={[form.formState.errors.variants?.[variantIndex]?.sku]} />
+            </Field>
+            <Field>
+              <FieldLabel htmlFor={`variant-${variantIndex}-barcode`}>
+                {t('admin.fields.variant.barcode.label')}
+              </FieldLabel>
+              <Input
+                id={`variant-${variantIndex}-barcode`}
+                {...form.register(`variants.${variantIndex}.barcode`)}
+              />
+              <FieldError errors={[form.formState.errors.variants?.[variantIndex]?.barcode]} />
+            </Field>
+          </FormSection>
+
+          <FormSection title={t('admin.fields.shipping.label')}>
+            <div className="grid grid-cols-[1fr_120px] gap-3">
+              <Field>
+                <FieldLabel htmlFor={`variant-${variantIndex}-weight`}>
+                  {t('admin.fields.variant.weight.label')}
+                </FieldLabel>
+                <Input
+                  id={`variant-${variantIndex}-weight`}
+                  type="number"
+                  step="0.01"
+                  min="0"
+                  {...form.register(`variants.${variantIndex}.weight`)}
+                />
+              </Field>
+              <Field>
+                <FieldLabel htmlFor={`variant-${variantIndex}-weight-unit`}>
+                  {t('admin.fields.variant.weight_unit.label')}
+                </FieldLabel>
+                <Controller
+                  name={`variants.${variantIndex}.weight_unit`}
+                  control={form.control}
+                  render={({ field }) => (
+                    <Select
+                      value={field.value ?? ''}
+                      onValueChange={(v) => field.onChange(v || null)}
+                    >
+                      <SelectTrigger id={`variant-${variantIndex}-weight-unit`} className="w-full">
+                        <SelectValue>{(v) => (v as string) || '—'}</SelectValue>
+                      </SelectTrigger>
+                      <SelectContent>
+                        {WEIGHT_UNITS.map((u) => (
+                          <SelectItem key={u} value={u}>
+                            {u}
+                          </SelectItem>
+                        ))}
+                      </SelectContent>
+                    </Select>
+                  )}
+                />
+              </Field>
+            </div>
+
+            <div className="grid grid-cols-[1fr_1fr_1fr_120px] gap-3">
+              <Field>
+                <FieldLabel htmlFor={`variant-${variantIndex}-height`}>
+                  {t('admin.fields.variant.height.label')}
+                </FieldLabel>
+                <Input
+                  id={`variant-${variantIndex}-height`}
+                  type="number"
+                  step="0.01"
+                  min="0"
+                  {...form.register(`variants.${variantIndex}.height`)}
+                />
+              </Field>
+              <Field>
+                <FieldLabel htmlFor={`variant-${variantIndex}-width`}>
+                  {t('admin.fields.variant.width.label')}
+                </FieldLabel>
+                <Input
+                  id={`variant-${variantIndex}-width`}
+                  type="number"
+                  step="0.01"
+                  min="0"
+                  {...form.register(`variants.${variantIndex}.width`)}
+                />
+              </Field>
+              <Field>
+                <FieldLabel htmlFor={`variant-${variantIndex}-depth`}>
+                  {t('admin.fields.variant.depth.label')}
+                </FieldLabel>
+                <Input
+                  id={`variant-${variantIndex}-depth`}
+                  type="number"
+                  step="0.01"
+                  min="0"
+                  {...form.register(`variants.${variantIndex}.depth`)}
+                />
+              </Field>
+              <Field>
+                <FieldLabel htmlFor={`variant-${variantIndex}-dim-unit`}>
+                  {t('admin.fields.variant.dimensions_unit.label')}
+                </FieldLabel>
+                <Controller
+                  name={`variants.${variantIndex}.dimensions_unit`}
+                  control={form.control}
+                  render={({ field }) => (
+                    <Select
+                      value={field.value ?? ''}
+                      onValueChange={(v) => field.onChange(v || null)}
+                    >
+                      <SelectTrigger id={`variant-${variantIndex}-dim-unit`} className="w-full">
+                        <SelectValue>{(v) => (v as string) || '—'}</SelectValue>
+                      </SelectTrigger>
+                      <SelectContent>
+                        {DIMENSION_UNITS.map((u) => (
+                          <SelectItem key={u} value={u}>
+                            {u}
+                          </SelectItem>
+                        ))}
+                      </SelectContent>
+                    </Select>
+                  )}
+                />
+              </Field>
+            </div>
+          </FormSection>
+
+          {cartonTypesSupported && (
+            <FormSection title={t('admin.products.variants.sheet.packing')}>
+              <p className="text-sm text-muted-foreground">
+                {t('admin.products.variants.sheet.packing_help')}
+              </p>
+              {knownEmpty && (
+                <Alert variant="info">
+                  <InfoIcon />
+                  <AlertDescription>
+                    <Trans
+                      i18nKey="admin.products.variants.sheet.no_carton_types"
+                      components={{
+                        settingsLink: (
+                          <Link
+                            to={`/${tenantId}/settings/package-types` as never}
+                            className="underline underline-offset-3"
+                          />
+                        ),
+                      }}
+                    />
+                  </AlertDescription>
+                </Alert>
+              )}
+              <Field>
+                <FieldLabel htmlFor={`variant-${variantIndex}-carton`}>
+                  {t('admin.fields.variant.carton_package_type_id.label')}
+                </FieldLabel>
+                <Controller
+                  name={`variants.${variantIndex}.carton_package_type_id`}
+                  control={form.control}
+                  render={({ field }) => (
+                    <Select
+                      value={field.value ?? ''}
+                      onValueChange={(value) => field.onChange(value || null)}
+                    >
+                      <SelectTrigger
+                        id={`variant-${variantIndex}-carton`}
+                        className="w-full"
+                        disabled={!hasCartonTypes}
+                      >
+                        <SelectValue
+                          placeholder={t('admin.products.variants.sheet.carton_placeholder')}
+                        >
+                          {(value) =>
+                            cartonTypes.find((carton) => carton.id === value)?.name ??
+                            t('admin.products.variants.sheet.carton_placeholder')
+                          }
+                        </SelectValue>
+                      </SelectTrigger>
+                      <SelectContent>
+                        {cartonTypes.map((carton) => (
+                          <SelectItem key={carton.id} value={carton.id}>
+                            <span className="flex w-full items-baseline justify-between gap-4">
+                              <span>{carton.name}</span>
+                              {/* Two cartons named alike are told apart by their
+                                  size, and the merchant is about to state what a
+                                  packed one weighs. */}
+                              {cartonSize(carton) && (
+                                <span className="text-xs text-muted-foreground tabular-nums">
+                                  {cartonSize(carton)}
+                                </span>
+                              )}
+                            </span>
+                          </SelectItem>
+                        ))}
+                      </SelectContent>
+                    </Select>
+                  )}
+                />
+                <FieldDescription>
+                  {t('admin.fields.variant.carton_package_type_id.help')}
+                </FieldDescription>
+              </Field>
+              <div className="grid grid-cols-2 gap-3">
+                <Field>
+                  <FieldLabel htmlFor={`variant-${variantIndex}-carton-weight`}>
+                    {t('admin.fields.variant.carton_weight.label')}
+                  </FieldLabel>
+                  {/* Cleared means unmeasured, not zero — and a zero carton
+                      weight is refused, so coercing a blank to 0 would leave
+                      the merchant unable to undo their own entry. */}
+                  <Controller
+                    name={`variants.${variantIndex}.carton_weight`}
+                    control={form.control}
+                    render={({ field }) => (
+                      <InputGroup>
+                        <InputGroupInput
+                          id={`variant-${variantIndex}-carton-weight`}
+                          type="number"
+                          step="0.01"
+                          min="0"
+                          disabled={!hasCartonTypes}
+                          value={field.value ?? ''}
+                          onChange={(event) => {
+                            const parsed = Number(event.target.value)
+                            field.onChange(
+                              event.target.value === '' || Number.isNaN(parsed) ? null : parsed,
+                            )
+                          }}
+                        />
+                        {/* The same scale the variant's own weight is in —
+                            there is one weight unit per variant, so saying
+                            which beats offering a second selector. */}
+                        <InputGroupAddon align="inline-end">
+                          <InputGroupText>{cartonWeightUnit}</InputGroupText>
+                        </InputGroupAddon>
+                      </InputGroup>
+                    )}
+                  />
+                  <FieldDescription>
+                    {t('admin.fields.variant.carton_weight.help')}
+                  </FieldDescription>
+                </Field>
+                <Field>
+                  <FieldLabel htmlFor={`variant-${variantIndex}-cartons-per-pallet`}>
+                    {t('admin.fields.variant.cartons_per_pallet.label')}
+                  </FieldLabel>
+                  <Input
+                    id={`variant-${variantIndex}-cartons-per-pallet`}
+                    type="number"
+                    min="1"
+                    step="1"
+                    disabled={!hasCartonTypes}
+                    {...form.register(`variants.${variantIndex}.cartons_per_pallet`)}
+                  />
+                  <FieldError
+                    errors={[form.formState.errors.variants?.[variantIndex]?.cartons_per_pallet]}
+                  />
+                </Field>
+              </div>
+            </FormSection>
+          )}
+
+          <FormSection title={t('admin.products.variants.sheet.customs')}>
+            <p className="text-sm text-muted-foreground">
+              {t('admin.products.variants.sheet.customs_help')}
+            </p>
+            <div className="grid grid-cols-2 gap-3">
+              <Field>
+                <FieldLabel htmlFor={`variant-${variantIndex}-hs-code`}>
+                  {t('admin.fields.variant.hs_code.label')}
+                </FieldLabel>
+                <Input
+                  id={`variant-${variantIndex}-hs-code`}
+                  inputMode="numeric"
+                  placeholder={t('admin.fields.variant.hs_code.placeholder')}
+                  {...form.register(`variants.${variantIndex}.hs_code`, {
+                    setValueAs: normalizeHsCode,
+                  })}
+                />
+                <FieldError errors={[form.formState.errors.variants?.[variantIndex]?.hs_code]} />
+              </Field>
+              <Field>
+                <FieldLabel htmlFor={`variant-${variantIndex}-country-of-origin`}>
+                  {t('admin.fields.variant.country_of_origin.label')}
+                </FieldLabel>
+                <Controller
+                  name={`variants.${variantIndex}.country_of_origin`}
+                  control={form.control}
+                  render={({ field }) => (
+                    <CountryCombobox
+                      id={`variant-${variantIndex}-country-of-origin`}
+                      value={field.value ?? null}
+                      onValueChange={(iso) => field.onChange(iso || null)}
+                    />
+                  )}
+                />
+                <FieldError
+                  errors={[form.formState.errors.variants?.[variantIndex]?.country_of_origin]}
+                />
+              </Field>
+            </div>
+            <Field>
+              <FieldLabel htmlFor={`variant-${variantIndex}-customs-description`}>
+                {t('admin.fields.variant.customs_description.label')}
+              </FieldLabel>
+              <Input
+                id={`variant-${variantIndex}-customs-description`}
+                placeholder={t('admin.fields.variant.customs_description.placeholder')}
+                {...form.register(`variants.${variantIndex}.customs_description`, {
+                  setValueAs: normalizeCustomsDescription,
+                })}
+              />
+              <FieldError
+                errors={[form.formState.errors.variants?.[variantIndex]?.customs_description]}
+              />
+            </Field>
+          </FormSection>
+
+          <FormSection title={t('admin.products.variants.sheet.ordering')}>
+            <p className="text-sm text-muted-foreground">
+              {t('admin.products.variants.sheet.ordering_help')}
+            </p>
+            <div className="grid grid-cols-2 gap-3">
+              <Field>
+                <FieldLabel htmlFor={`variant-${variantIndex}-moq`}>
+                  {t('admin.fields.variant.minimum_order_quantity.label')}
+                </FieldLabel>
+                <Input
+                  id={`variant-${variantIndex}-moq`}
+                  type="number"
+                  min="1"
+                  step="1"
+                  placeholder={t('admin.fields.variant.minimum_order_quantity.placeholder')}
+                  {...form.register(`variants.${variantIndex}.minimum_order_quantity`)}
+                />
+                <FieldError
+                  errors={[form.formState.errors.variants?.[variantIndex]?.minimum_order_quantity]}
+                />
+              </Field>
+              <Field>
+                <FieldLabel htmlFor={`variant-${variantIndex}-order-multiple`}>
+                  {t('admin.fields.variant.order_multiple.label')}
+                </FieldLabel>
+                <Input
+                  id={`variant-${variantIndex}-order-multiple`}
+                  type="number"
+                  min="1"
+                  step="1"
+                  placeholder={t('admin.fields.variant.order_multiple.placeholder')}
+                  {...form.register(`variants.${variantIndex}.order_multiple`)}
+                />
+                <FieldError
+                  errors={[form.formState.errors.variants?.[variantIndex]?.order_multiple]}
+                />
+              </Field>
+              <Field>
+                <FieldLabel htmlFor={`variant-${variantIndex}-purchase-unit`}>
+                  {t('admin.fields.variant.purchase_unit.label')}
+                </FieldLabel>
+                <Controller
+                  name={`variants.${variantIndex}.purchase_unit`}
+                  control={form.control}
+                  render={({ field }) => (
+                    <Select
+                      value={field.value ?? ''}
+                      onValueChange={(v) => field.onChange(v || null)}
+                    >
+                      <SelectTrigger
+                        id={`variant-${variantIndex}-purchase-unit`}
+                        className="w-full"
+                      >
+                        <SelectValue>
+                          {(v) =>
+                            v
+                              ? t(`admin.fields.variant.purchase_unit.options.${v as string}`)
+                              : t('admin.fields.variant.purchase_unit.options.unit')
+                          }
+                        </SelectValue>
+                      </SelectTrigger>
+                      <SelectContent>
+                        {PURCHASE_UNITS.map((unit) => (
+                          <SelectItem key={unit} value={unit}>
+                            {t(`admin.fields.variant.purchase_unit.options.${unit}`)}
+                          </SelectItem>
+                        ))}
+                      </SelectContent>
+                    </Select>
+                  )}
+                />
+              </Field>
+              <Field>
+                <FieldLabel htmlFor={`variant-${variantIndex}-units-per-carton`}>
+                  {t('admin.fields.variant.units_per_carton.label')}
+                </FieldLabel>
+                <Input
+                  id={`variant-${variantIndex}-units-per-carton`}
+                  type="number"
+                  min="1"
+                  step="1"
+                  {...form.register(`variants.${variantIndex}.units_per_carton`)}
+                />
+                <FieldError
+                  errors={[form.formState.errors.variants?.[variantIndex]?.units_per_carton]}
+                />
+              </Field>
+            </div>
+          </FormSection>
+
+          <FormSection title={t('admin.products.variants.sheet.availability')}>
+            <Field>
+              <div className="flex items-start justify-between gap-4">
+                <div className="flex flex-col">
+                  <FieldLabel
+                    htmlFor={`variant-${variantIndex}-preorderable`}
+                    className="cursor-pointer"
+                  >
+                    {t('admin.fields.variant.preorderable.label')}
+                  </FieldLabel>
+                  <span className="text-xs text-muted-foreground">
+                    {t('admin.fields.variant.preorderable.help')}
+                  </span>
+                </div>
+                <Controller
+                  name={`variants.${variantIndex}.preorderable`}
+                  control={form.control}
+                  render={({ field }) => (
+                    <Switch
+                      id={`variant-${variantIndex}-preorderable`}
+                      checked={!!field.value}
+                      onCheckedChange={field.onChange}
+                    />
+                  )}
+                />
+              </div>
+            </Field>
+
+            {variant.preorderable && (
+              <Field>
+                <FieldLabel>{t('admin.fields.variant.preorder_ships_at.label')}</FieldLabel>
+                <Controller
+                  control={form.control}
+                  name={`variants.${variantIndex}.preorder_ships_at`}
+                  render={({ field }) => (
+                    <StoreDatePicker
+                      value={field.value ?? null}
+                      onChange={(next) => field.onChange(next ?? null)}
+                      placeholder={t('admin.fields.variant.preorder_ships_at.placeholder')}
+                      includeTime
+                      inline
+                    />
+                  )}
+                />
+                <span className="text-xs text-muted-foreground">
+                  {t('admin.fields.variant.preorder_ships_at.help')}
+                </span>
+              </Field>
+            )}
+
+            <Field>
+              <FieldLabel htmlFor={`variant-${variantIndex}-backorder-limit`}>
+                {t('admin.fields.variant.backorder_limit.label')}
+              </FieldLabel>
+              <Controller
+                control={form.control}
+                name={`variants.${variantIndex}.backorder_limit`}
+                render={({ field }) => (
+                  <Input
+                    id={`variant-${variantIndex}-backorder-limit`}
+                    type="number"
+                    min="0"
+                    step="1"
+                    placeholder={t('admin.fields.variant.backorder_limit.placeholder')}
+                    value={field.value ?? ''}
+                    onChange={(event) => {
+                      const parsed = Number(event.target.value)
+                      field.onChange(
+                        event.target.value === '' || Number.isNaN(parsed)
+                          ? null
+                          : Math.max(0, Math.trunc(parsed)),
+                      )
+                    }}
+                  />
+                )}
+              />
+              <span className="text-xs text-muted-foreground">
+                {t('admin.fields.variant.backorder_limit.help')}
+              </span>
+              <FieldError
+                errors={[form.formState.errors.variants?.[variantIndex]?.backorder_limit]}
+              />
+            </Field>
+          </FormSection>
+
+          {hasTaxCategories && (
+            <FormSection title={t('admin.fields.tax.label')}>
+              <Field>
+                <FieldLabel htmlFor={`variant-${variantIndex}-tax-cat`}>
+                  {t('admin.fields.tax_category_id.label')}
+                </FieldLabel>
+                <Controller
+                  name={`variants.${variantIndex}.tax_category_id`}
+                  control={form.control}
+                  render={({ field }) => (
+                    <Select
+                      value={field.value ?? ''}
+                      onValueChange={(v) => field.onChange(v || null)}
+                    >
+                      <SelectTrigger id={`variant-${variantIndex}-tax-cat`} className="w-full">
+                        <SelectValue
+                          placeholder={t('admin.products.variants.sheet.tax_category_placeholder')}
+                        >
+                          {(v) =>
+                            taxCategories.find((c) => c.id === v)?.name ??
+                            t('admin.products.variants.sheet.tax_category_placeholder')
+                          }
+                        </SelectValue>
+                      </SelectTrigger>
+                      <SelectContent>
+                        {taxCategories.map((cat) => (
+                          <SelectItem key={cat.id} value={cat.id}>
+                            {cat.name}
+                          </SelectItem>
+                        ))}
+                      </SelectContent>
+                    </Select>
+                  )}
+                />
+              </Field>
+            </FormSection>
+          )}
+        </div>
+
+        <SheetFooter>
+          <Button type="button" variant="outline" onClick={handleCancel}>
+            {t('admin.actions.cancel')}
+          </Button>
+          <Button type="button" onClick={handleDone}>
+            {t('admin.actions.done')}
+          </Button>
+        </SheetFooter>
+      </SheetContent>
+    </Sheet>
+  )
+}