@phillips/seldon 1.240.1 → 1.242.0

package/README.md

@@ -44,3 +44,33 @@ import { Button } from '@phillips/seldon';
 ### Contributing Guidelines
 
 Before you start contributing to this project please check out our guidelines found [here](/src/docs/CONTRIBUTING.md)
+
+### Git worktrees (`new-worktree.sh`)
+
+You can use Git [worktrees](https://git-scm.com/docs/git-worktree) to keep multiple branches checked out at once (for example when reviewing a PR or spiking a change) without re-running all setup steps every time.
+
+Use the `scripts/new-worktree.sh` helper to create a new worktree and copy ignored files (like `.env`) into the new directory:
+
+```bash
+# New worktree at ../my-feature using the current branch
+./scripts/new-worktree.sh ../my-feature
+
+# New worktree on an existing branch
+./scripts/new-worktree.sh ../my-feature feature-branch
+
+# New worktree with a new branch (e.g. from main)
+./scripts/new-worktree.sh ../my-feature -b new-branch main
+
+# Optionally copy node_modules instead of running npm install in the new worktree
+COPY_NODE_MODULES=1 ./scripts/new-worktree.sh ../my-feature
+```
+
+The script runs `git worktree add`, copies ignored config files into the new directory, runs `npm install`, and then opens the new worktree in Cursor or VS Code.
+
+To remove a worktree, you can run the following command:
+
+```bash
+git worktree remove ../my-feature
+```
+
+This will remove the worktree from the git repository.

package/dist/_virtual/index7.js

@@ -1,4 +1,5 @@
-var p = { exports: {} };
+import { __require as r } from "../node_modules/ics/dist/index.js";
+var i = r();
 export {
-  p as __module
+  i as d
 };

package/dist/_virtual/index8.js

@@ -1,5 +1,4 @@
-import { __require as r } from "../node_modules/ics/dist/index.js";
-var i = r();
+var p = { exports: {} };
 export {
-  i as d
+  p as __module
 };

package/dist/components/Accordion/AccordionItem.js

@@ -1,55 +1,55 @@
 import { jsxs as N, jsx as t } from "react/jsx-runtime";
 import { Item as A, Trigger as _, Content as k } from "../../node_modules/@radix-ui/react-accordion/dist/index.js";
 import C from "../../_virtual/index.js";
-import { forwardRef as v, useRef as w, useCallback as T } from "react";
-import $ from "../Icon/Icon.js";
-import { getCommonProps as y, px as R } from "../../utils/index.js";
+import { forwardRef as v, useRef as w, useCallback as y } from "react";
+import I from "../Icon/Icon.js";
+import { getCommonProps as T, px as R } from "../../utils/index.js";
 import { AccordionItemVariant as d } from "./types.js";
-import { getIconClasses as b } from "./utils.js";
-import { TextVariants as u } from "../Text/types.js";
+import { getIconClasses as $ } from "./utils.js";
+import { TextVariants as h } from "../Text/types.js";
 import x from "../Text/Text.js";
 const S = ({
   children: e,
   className: n,
   baseClassName: o,
   disable: r,
-  id: i,
-  variant: c = d.md,
+  id: c,
+  variant: i = d.md,
   onOpen: l,
   onClose: m
 }) => {
-  const h = w(null), s = r, p = c === d.md ? u.labelMedium : u.labelSmall, f = /* @__PURE__ */ t("div", { children: /* @__PURE__ */ t(
-    $,
+  const u = w(null), s = r, p = i === d.md ? h.labelMedium : h.labelSmall, f = /* @__PURE__ */ t("div", { children: /* @__PURE__ */ t(
+    I,
     {
       icon: "Lock",
       height: 24,
       width: 24,
-      className: b(o, c, "lock"),
-      "data-testid": `${i}-lockedIcon`,
+      className: $(o, i, "lock"),
+      "data-testid": `${c}-lockedIcon`,
       "aria-hidden": !0
     }
   ) }), g = /* @__PURE__ */ t("div", { children: /* @__PURE__ */ t(
-    $,
+    I,
     {
       icon: "Add",
       height: 24,
       width: 24,
-      className: b(o, c, "plus"),
-      "data-testid": `${i}-plusIcon`,
+      className: $(o, i, "plus"),
+      "data-testid": `${c}-plusIcon`,
       "aria-hidden": !0
     }
-  ) }), I = /* @__PURE__ */ t("div", { children: /* @__PURE__ */ t(
-    $,
+  ) }), b = /* @__PURE__ */ t("div", { children: /* @__PURE__ */ t(
+    I,
     {
       icon: "Subtract",
       height: 24,
       width: 24,
-      className: b(o, c, "minus"),
-      "data-testid": `${i}-minusIcon`,
+      className: $(o, i, "minus"),
+      "data-testid": `${c}-minusIcon`,
       "aria-hidden": !0
     }
-  ) }), a = T(() => {
-    h.current?.getAttribute("data-state") === "closed" ? l?.() : m?.();
+  ) }), a = y(() => {
+    u.current?.getAttribute("data-state") === "closed" ? l?.() : m?.();
   }, [l, m]);
   return /* @__PURE__ */ t(
     _,
@@ -57,13 +57,13 @@ const S = ({
       "data-disabled": r,
       asChild: !0,
       className: C(o, { [`${o}--hoverable`]: !r }, n),
-      ref: h,
+      ref: u,
       onClick: a,
-      children: /* @__PURE__ */ N("div", { "data-testid": `${i}-trigger`, children: [
+      children: /* @__PURE__ */ N("button", { type: "button", "data-testid": `${c}-trigger`, children: [
         /* @__PURE__ */ t(x, { className: `${R}-accordion-item-label__text`, variant: p, children: e }),
         s && f,
         !s && g,
-        !s && I
+        !s && b
       ] })
     }
   );
@@ -72,8 +72,8 @@ const S = ({
   baseClassName: n,
   disable: o,
   hasTransition: r,
-  className: i,
-  variant: c = d.md
+  className: c,
+  variant: i = d.md
 }) => o && e ? /* @__PURE__ */ t("div", { className: `${n}__content--locked`, children: e }) : /* @__PURE__ */ t(
   k,
   {
@@ -81,9 +81,9 @@ const S = ({
     className: C(
       `${n}__content`,
       { [`${n}--transition`]: r },
-      i
+      c
     ),
-    children: /* @__PURE__ */ t(x, { variant: c === d.md ? u.bodyMedium : u.bodySmall, children: e })
+    children: /* @__PURE__ */ t(x, { variant: i === d.md ? h.bodyMedium : h.bodySmall, children: e })
   }
 ), j = v(
   ({
@@ -91,24 +91,24 @@ const S = ({
     variant: n = d.md,
     id: o,
     label: r,
-    isLastItem: i,
-    hasTransition: c = !1,
+    isLastItem: c,
+    hasTransition: i = !1,
     children: l,
     className: m,
-    transitionTimeInMs: h = 250,
+    transitionTimeInMs: u = 250,
     onOpen: s,
     onClose: p,
     ...f
   }, g) => {
-    const { className: I } = y({ id: o }, "Accordion"), a = `${I}-item`;
+    const { className: b } = T({ id: o }, "Accordion"), a = `${b}-item`;
     return /* @__PURE__ */ N(
       A,
       {
         disabled: e,
         value: o,
-        style: { "--seldon-accordion-transition-time": `${h}ms` },
+        style: { "--seldon-accordion-transition-time": `${u}ms` },
         className: C(a, m, {
-          [`${a}__border-bottom`]: !i
+          [`${a}__border-bottom`]: !c
         }),
         ref: g,
         ...f,
@@ -129,7 +129,7 @@ const S = ({
             V,
             {
               disable: e,
-              hasTransition: c,
+              hasTransition: i,
               variant: n,
               baseClassName: a,
               children: /* @__PURE__ */ t("div", { className: "radix-accordion-content", children: l })

package/dist/components/AddToCalendar/calendarLinks.js

@@ -1,4 +1,4 @@
-import { d as u } from "../../_virtual/index8.js";
+import { d as u } from "../../_virtual/index7.js";
 import { isValid as p } from "../../node_modules/date-fns/isValid.js";
 import { toZonedTime as m } from "../../node_modules/date-fns-tz/dist/esm/toZonedTime/index.js";
 import { addHours as i } from "../../node_modules/date-fns/addHours.js";

package/dist/components/Button/Button.js

@@ -1,53 +1,53 @@
-import { jsxs as j, Fragment as C, jsx as n } from "react/jsx-runtime";
+import { jsxs as V, Fragment as C, jsx as t } from "react/jsx-runtime";
 import N from "../../_virtual/index.js";
-import { getCommonProps as v, px as w } from "../../utils/index.js";
-import { ButtonVariants as l, ButtonSizes as y } from "./types.js";
+import { getCommonProps as j, px as w } from "../../utils/index.js";
+import { ButtonVariants as m, ButtonSizes as v } from "./types.js";
 import { forwardRef as H, useState as g } from "react";
-import { LinkVariants as s } from "../Link/types.js";
+import { LinkVariants as a } from "../Link/types.js";
 import F from "../Link/Link.js";
 const I = H(
   ({
-    variant: t = l.primary,
-    size: k = y.default,
+    variant: o = m.primary,
+    size: k = v.default,
     children: u,
     className: c,
     isIconLast: f = !1,
-    type: L = "button",
-    isDisabled: a = !1,
-    href: o,
+    type: y = "button",
+    isDisabled: l = !1,
+    href: n,
     target: d,
     prefetch: p = "none",
-    isSkeletonLoading: i,
-    linkSize: M = "md",
-    ...m
+    isSkeletonLoading: s,
+    linkSize: L = "md",
+    ...i
   }, $) => {
-    const { className: e, ...b } = v(m, "Button"), [P, V] = g(!1);
-    if (o || t === l.link) {
-      const B = () => /* @__PURE__ */ n(C, { children: /* @__PURE__ */ n("link", { "data-testid": "prefetch-link", rel: "prefetch", href: o }) });
-      let r = s.linkMedium;
-      switch (M) {
+    const { className: e, ...b } = j(i, "Button"), [M, P] = g(!1);
+    if (n) {
+      const B = () => /* @__PURE__ */ t(C, { children: /* @__PURE__ */ t("link", { "data-testid": "prefetch-link", rel: "prefetch", href: n }) });
+      let r = a.linkMedium;
+      switch (L) {
         case "lg":
-          r = s.linkLarge;
+          r = a.linkLarge;
           break;
         case "md":
-          r = s.linkMedium;
+          r = a.linkMedium;
           break;
         case "sm":
-          r = s.linkSmall;
+          r = a.linkSmall;
           break;
       }
-      return /* @__PURE__ */ j(C, { children: [
-        /* @__PURE__ */ n(
+      return /* @__PURE__ */ V(C, { children: [
+        /* @__PURE__ */ t(
           F,
           {
             ...b,
             variant: r,
             ref: $,
-            href: o,
+            href: n,
             className: N(
               `${e}`,
-              { [`${e}--${t}`]: t !== l.link },
-              { [`${e}--link`]: t === l.link },
+              { [`${e}--${o}`]: o !== m.link },
+              { [`${e}--link`]: o === m.link },
               {
                 [`${e}--icon-last`]: f
               },
@@ -56,37 +56,37 @@ const I = H(
             target: d,
             rel: d === "_blank" ? "noopener noreferrer" : void 0,
             onMouseOver: () => {
-              a || V(!0);
+              l || P(!0);
             },
             onClick: (x) => {
-              a ? x.preventDefault() : m.onClick?.(x);
+              l ? x.preventDefault() : i.onClick?.(x);
             },
             children: u
           }
         ),
-        p === "intent" && o && P && /* @__PURE__ */ n(B, {}),
-        p === "render" && o && /* @__PURE__ */ n(B, {})
+        p === "intent" && n && M && /* @__PURE__ */ t(B, {}),
+        p === "render" && n && /* @__PURE__ */ t(B, {})
       ] });
     } else
-      return /* @__PURE__ */ n(
+      return /* @__PURE__ */ t(
         "button",
         {
           ...b,
           ref: $,
-          type: L,
-          "aria-busy": i,
+          type: y,
+          "aria-busy": s,
           className: N(
             `${e}`,
-            `${e}--${t}`,
-            { [`${e}--${k}`]: k !== y.default },
+            `${e}--${o}`,
+            { [`${e}--${k}`]: k !== v.default },
             {
               [`${e}--icon-last`]: f,
-              [`${w}-skeleton`]: i
+              [`${w}-skeleton`]: s
             },
             c
           ),
-          disabled: i || a,
-          ...m,
+          disabled: s || l,
+          ...i,
           children: u
         }
       );

package/dist/components/Button/types.d.ts

@@ -2,6 +2,7 @@ export declare enum ButtonVariants {
     primary = "primary",
     secondary = "secondary",
     tertiary = "tertiary",
+    destructive = "destructive",
     link = "link"
 }
 export declare enum ButtonSizes {

package/dist/components/Button/types.js

@@ -1,4 +1,4 @@
-var e = /* @__PURE__ */ ((r) => (r.primary = "primary", r.secondary = "secondary", r.tertiary = "tertiary", r.link = "link", r))(e || {}), l = /* @__PURE__ */ ((r) => (r.default = "default", r.small = "small", r))(l || {});
+var e = /* @__PURE__ */ ((r) => (r.primary = "primary", r.secondary = "secondary", r.tertiary = "tertiary", r.destructive = "destructive", r.link = "link", r))(e || {}), l = /* @__PURE__ */ ((r) => (r.default = "default", r.small = "small", r))(l || {});
 export {
   l as ButtonSizes,
   e as ButtonVariants

package/dist/components/ExitGateCard/ExitGateCard.js

@@ -9,7 +9,7 @@ import v from "../Link/Link.js";
 import G from "../SeldonImage/SeldonImage.js";
 import { TextVariants as p } from "../Text/types.js";
 import a from "../Text/Text.js";
-const j = $(
+const b = $(
   ({
     className: f,
     imageSrc: r,
@@ -22,9 +22,9 @@ const j = $(
     variant: _ = E.secondary,
     altText: u = "Exit Gate Card Image",
     ...c
-  }, N) => {
-    const { className: t, ...x } = g(c, "ExitGateCard");
-    return /* @__PURE__ */ d("article", { ...x, className: h(t, f), ...c, ref: N, children: [
+  }, x) => {
+    const { className: t, ...N } = g(c, "ExitGateCard");
+    return /* @__PURE__ */ d("article", { ...N, className: h(t, f), ...c, ref: x, children: [
       r ? /* @__PURE__ */ e(
         G,
         {
@@ -39,12 +39,12 @@ const j = $(
         n ? /* @__PURE__ */ e(a, { variant: p.labelMedium, element: "span", className: `${t}__content-label`, children: n }) : null,
         m ? /* @__PURE__ */ e(a, { variant: p.headingLarge, element: "span", className: `${t}__content-title`, children: m }) : null,
         o ? /* @__PURE__ */ e(a, { element: "span", className: `${t}__content-description`, children: o }) : null,
-        (s || i) && l ? /* @__PURE__ */ e(i, { href: l, children: /* @__PURE__ */ e(C, { variant: _, className: `${t}__content-link`, children: s }) }) : null
+        (s || i) && l ? /* @__PURE__ */ e(i, { href: l, children: /* @__PURE__ */ e(C, { variant: _, tabIndex: -1, className: `${t}__content-link`, children: s }) }) : null
       ] })
     ] });
   }
 );
-j.displayName = "ExitGateCard";
+b.displayName = "ExitGateCard";
 export {
-  j as default
+  b as default
 };

package/dist/components/Tags/Tags.js

@@ -1,13 +1,13 @@
-import { jsxs as n, jsx as a } from "react/jsx-runtime";
+import { jsxs as m, jsx as a } from "react/jsx-runtime";
 import N, { forwardRef as b } from "react";
 import d from "../../_virtual/index.js";
-import { getCommonProps as x, px as r } from "../../utils/index.js";
+import { getCommonProps as v, px as r } from "../../utils/index.js";
 import f from "../Button/Button.js";
 import { ButtonVariants as p } from "../Button/types.js";
 import $ from "../Icon/Icon.js";
 import { TextVariants as u } from "../Text/types.js";
 import g from "../Text/Text.js";
-const L = ({ id: s, className: t, onRemove: e, label: o, removeText: i = "Remove" }) => /* @__PURE__ */ n(
+const P = ({ id: s, className: t, onRemove: e, label: o, removeText: i = "Remove" }) => /* @__PURE__ */ m(
   f,
   {
     className: d(`${r}-tag`, `${r}-button`, t),
@@ -19,25 +19,24 @@ const L = ({ id: s, className: t, onRemove: e, label: o, removeText: i = "Remove
       /* @__PURE__ */ a("div", { className: `${r}-tag__button`, "data-testid": `${s}-item-close-button`, children: /* @__PURE__ */ a($, { icon: "CloseX", height: 8, width: 8, color: "currentColor", className: `${r}-tag__icon` }) })
     ]
   }
-), v = b(
+), x = b(
   ({ className: s, children: t, clearAllLabel: e = "Clear All", onClear: o, ...i }, h) => {
-    const l = "tags-list", { className: C, ...y } = x(i, "TagsList"), { id: m } = i;
-    return /* @__PURE__ */ n(
+    const l = "tags-list", { className: C, ...y } = v(i, "TagsList"), { id: n } = i;
+    return /* @__PURE__ */ m(
       "ul",
       {
         className: d(`${r}-${l}`, C, s),
         ...y,
         ...i,
-        tabIndex: 0,
-        "data-testid": `${l}-${m}`,
+        "data-testid": `${l}-${n}`,
         ref: h,
         children: [
           Array.isArray(t) ? N.Children.map(t, (c) => /* @__PURE__ */ a("li", { children: c }, c.props.id)) : t,
-          Array.isArray(t) && t.length > 0 && /* @__PURE__ */ a("li", { children: /* @__PURE__ */ n(
+          Array.isArray(t) && t.length > 0 && /* @__PURE__ */ a("li", { children: /* @__PURE__ */ m(
             f,
             {
               onClick: o,
-              "data-testid": `${m}-clear-all-button`,
+              "data-testid": `${n}-clear-all-button`,
               className: `${r}-${l}--clear`,
               "aria-label": e,
               variant: p.tertiary,
@@ -52,8 +51,8 @@ const L = ({ id: s, className: t, onRemove: e, label: o, removeText: i = "Remove
     );
   }
 );
-v.displayName = "Tags";
+x.displayName = "Tags";
 export {
-  L as Tag,
-  v as default
+  P as Tag,
+  x as default
 };

package/dist/node_modules/prop-types/index.js

@@ -1,4 +1,4 @@
-import { __module as r } from "../../_virtual/index7.js";
+import { __module as r } from "../../_virtual/index8.js";
 import { __require as s } from "./node_modules/react-is/index.js";
 import { __require as t } from "./factoryWithTypeCheckers.js";
 import { __require as p } from "./factoryWithThrowingShims.js";

package/dist/patterns/Social/Social.js

@@ -1,29 +1,29 @@
-import { jsxs as f, jsx as a } from "react/jsx-runtime";
-import p from "../../_virtual/index.js";
-import { getCommonProps as u } from "../../utils/index.js";
-import N from "../../components/Button/Button.js";
-import { forwardRef as h } from "react";
-import { ButtonVariants as v } from "../../components/Button/types.js";
-import { TextVariants as t } from "../../components/Text/types.js";
-import m from "../../components/Text/Text.js";
-const x = h(
+import { jsxs as c, jsx as a } from "react/jsx-runtime";
+import f from "../../_virtual/index.js";
+import { getCommonProps as p } from "../../utils/index.js";
+import d from "../../components/Button/Button.js";
+import { forwardRef as u } from "react";
+import { ButtonVariants as N } from "../../components/Button/types.js";
+import { TextVariants as h } from "../../components/Text/types.js";
+import x from "../../components/Text/Text.js";
+const S = u(
   ({
-    className: e,
-    children: i,
-    titleText: s = "Never miss a moment",
-    buttonText: n = "Subscribe to our newsletter",
-    onSubscribeClick: l,
+    className: t,
+    children: m,
+    titleText: e = "Never miss a moment",
+    buttonText: s = "Subscribe to our newsletter",
+    onSubscribeClick: i,
     ...o
-  }, c) => {
-    const { className: r, ...d } = u(o, "Social");
-    return /* @__PURE__ */ f("div", { ref: c, ...d, className: p(r, e), ...o, children: [
-      /* @__PURE__ */ a(m, { variant: t.headingSmall, className: `${r}__header`, children: s }),
-      /* @__PURE__ */ a(N, { onClick: l, variant: v.tertiary, className: `${r}__button`, children: /* @__PURE__ */ a(m, { variant: t.linkMedium, children: n }) }),
-      i
+  }, n) => {
+    const { className: r, ...l } = p(o, "Social");
+    return /* @__PURE__ */ c("div", { ref: n, ...l, className: f(r, t), ...o, children: [
+      /* @__PURE__ */ a(x, { variant: h.headingSmall, className: `${r}__header`, children: e }),
+      /* @__PURE__ */ a(d, { onClick: i, variant: N.tertiary, className: `${r}__button`, children: s }),
+      m
     ] });
   }
 );
-x.displayName = "Social";
+S.displayName = "Social";
 export {
-  x as default
+  S as default
 };

package/dist/scss/_utils.scss

@@ -239,3 +239,13 @@
     }
   }
 }
+
+@mixin focus-ring($border-radius: 0, $outline-offset: 4px, $outline-width: 2px, $outline-color: $focus-default) {
+  @if $border-radius != 0 {
+    /* stylelint-disable-next-line declaration-property-value-allowed-list */
+    border-radius: $border-radius;
+  }
+
+  outline: $outline-width solid $outline-color;
+  outline-offset: $outline-offset;
+}

package/dist/scss/components/Accordion/_accordion.scss

@@ -98,20 +98,30 @@
 
 .#{$px}-accordion-item-label {
   align-items: center;
+
+  // Reset button styles to match div appearance
+  background: none;
+  border: none;
   display: flex;
   flex-direction: row;
+  font: inherit;
   justify-content: space-between;
   padding: $spacing-sm 0;
+  text-align: inherit;
+  width: 100%;
 
   &--hoverable {
     cursor: pointer;
 
-    &:hover,
-    &:focus-visible {
+    &:hover {
       opacity: 0.75;
     }
   }
 
+  &:focus-visible {
+    @include focus-ring;
+  }
+
   &__text {
     flex: auto;
     font-variation-settings: 'wght' 600;

package/dist/scss/components/AddToCalendar/_addToCalendar.scss

@@ -23,8 +23,7 @@
     }
 
     &:focus-visible {
-      border-radius: $radius-xs;
-      outline: 1px solid $medium-gray;
+      @include focus-ring($border-radius: $radius-xs);
     }
   }
 

package/dist/scss/components/Breadcrumb/_breadcrumb.scss

@@ -27,8 +27,7 @@
       }
 
       &:focus-visible {
-        border-radius: $radius-xs;
-        outline: 1px solid $medium-gray;
+        @include focus-ring($border-radius: $radius-xs);
       }
     }