@stonedogcode/style 0.10.0 → 0.10.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stonedogcode/style",
-  "version": "0.10.0",
+  "version": "0.10.1",
   "description": "A Panda CSS design system: a themeable Panda preset plus the React components built on it.",
   "license": "Apache-2.0",
   "author": "StoneDogCode L.L.C.",

package/src/components/StyledIconButton.tsx

@@ -57,6 +57,28 @@ export interface StyledIconButtonProps extends HTMLStyledProps<"button"> {
   rel?: string;
 }
 
+/**
+ * Props this component used to accept and no longer honours (NEH-498).
+ *
+ * They are dropped rather than forwarded because everything else in `rest` is
+ * spread onto the element: left alone, `confirm={true}` reaches the DOM as an
+ * invalid attribute and React warns about each one in the consumer's console.
+ * The doc above already promised these were ignored — this is what makes that
+ * true.
+ *
+ * This is a migration seam with an end date. Delete it once no consumer passes
+ * any of them; none does today, which is why they were removed in the first
+ * place.
+ */
+const REMOVED_PROPS = [
+  "confirm",
+  "confirmTitle",
+  "confirmBody",
+  "onConfirm",
+  "loading",
+  "noBackground",
+] as const;
+
 /** Map the full vocabulary onto what the recipe can actually paint. */
 function toIconVariant(variant: AllowedVariant): IconButtonVariant {
   if (variant === "unstyled") return "ghost";
@@ -92,6 +114,10 @@ const StyledIconButton = React.forwardRef<HTMLButtonElement, StyledIconButtonPro
     // ignored.
     const { zIndex, style, className: incoming, ...restWithoutZIndex } = rest;
 
+    for (const prop of REMOVED_PROPS) {
+      delete (restWithoutZIndex as Record<string, unknown>)[prop];
+    }
+
     return (
       <StyledTooltip tooltip={tooltip} placement={placement}>
         <Element

package/src/components/StyledInputRadio.tsx

@@ -32,6 +32,14 @@ import { useResolvedVariant } from "../config/style-config";
  * radio rather than the group or the selection — reliably the wrong element.
  * It now points at the group container. Nothing was using it.
  *
+ * ## `onChange` is optional, and its absence means read-only (NEH-498)
+ *
+ * `checked` is always passed, so the inputs are controlled whether or not a
+ * handler is. Without one React reverts every click *and* warns once per radio
+ * in the consumer's console — a warning about nothing, which is the kind that
+ * teaches people to ignore the real ones. The inputs are marked `readOnly`
+ * instead, which is what the group actually is.
+ *
  * The container is a plain `<div>` rather than `StyledBox`, which is what the
  * original used. `StyledBox` wraps its children in an inner element unless told
  * not to, and that element would sit between the `radiogroup` and its radios —
@@ -121,7 +129,10 @@ const StyledInputRadio = React.forwardRef<HTMLDivElement, StyledInputRadioProps>
                 value={radio.value}
                 checked={value === radio.value}
                 disabled={radio.disabled}
-                onChange={onChange}
+                // A group with no `onChange` really is read-only — React reverts
+                // the click on a controlled input either way — so say so rather
+                // than leave React to warn about it in every consumer's console.
+                {...(onChange ? { onChange } : { readOnly: true })}
                 className={input}
               />
               <div className={control}>