@fabio.caffarello/react-design-system 3.2.0 → 3.3.1

package/dist/index.js

@@ -5487,6 +5487,40 @@ const Al = Me(
           "hover:bg-surface-hover",
           "focus:ring-line-focus",
           f("none", "p")
+        ),
+        // Textual call-to-action — brand-coloured text, underline on
+        // hover, no chrome (no surface, no border). Padding is zeroed
+        // out per size in compoundVariants so the bounding box hugs
+        // the text (height intrinsic). Issue #156.
+        //
+        // Focus ring: `focus:ring-line-focus` keeps the same focus
+        // family the other no-chrome variants use (ghost, outline,
+        // iconOnly). The base's `focus:ring-2` + `focus:ring-offset-2`
+        // produce a 2px ring 2px away from the text bounding box; the
+        // 2px offset is rendered in surface-base so the ring contrasts
+        // against surface (≥3:1 for WCAG 2.4.11, verified) rather than
+        // colliding with the brand-coloured text. The element keeps
+        // `getRadiusClass("md")` from the base, which is invisible
+        // without chrome but rounds the focus ring corners.
+        //
+        // hover:underline pairs with `underline-offset-4` so the
+        // underline that appears on hover sits clear of the descenders
+        // (WCAG-aligned with link best practice). At rest the link
+        // carries no underline — the brand colour alone signals
+        // affordance, matching the shadcn convention this variant is
+        // modelled on (see issue body).
+        //
+        // disabled: inherits opacity-50 + cursor-not-allowed from the
+        // base. We do NOT special-case `disabled:no-underline` because
+        // a disabled link should still receive the same visual
+        // treatment as a disabled chrome variant — the opacity and
+        // cursor signal the disabled state.
+        link: j(
+          "bg-transparent",
+          "text-fg-brand",
+          "underline-offset-4",
+          "hover:underline",
+          "focus:ring-line-focus"
         )
       },
       size: {
@@ -5526,6 +5560,27 @@ const Al = Me(
         variant: "iconOnly",
         size: "lg",
         class: j("h-12", "w-12", f("none", "p"))
+      },
+      // Link variant zeroes the size's padding via compoundVariants so
+      // the override runs AFTER the size block's `px-N`/`py-N` and
+      // twMerge picks the zero value (`cn` joins variant before
+      // compound). The size's typography and gap survive — the link's
+      // text scales with size and icons still get gap-N when present.
+      // Issue #156.
+      {
+        variant: "link",
+        size: "sm",
+        class: j(f("none", "px"), f("none", "py"))
+      },
+      {
+        variant: "link",
+        size: "md",
+        class: j(f("none", "px"), f("none", "py"))
+      },
+      {
+        variant: "link",
+        size: "lg",
+        class: j(f("none", "px"), f("none", "py"))
       }
     ],
     defaultVariants: {
@@ -8384,7 +8439,7 @@ const rs = Le(function(d) {
       role: b,
       tabIndex: x,
       onClick: a,
-      onKeyDown: g,
+      onKeyDown: p ? g : void 0,
       "aria-label": o,
       "aria-labelledby": i
     }, c), {