@phillips/seldon 1.240.1 → 1.241.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/index10.js

@@ -1,4 +1,4 @@
-var r = {};
+var e = {};
 export {
-  r as __exports
+  e as __exports
 };

package/dist/_virtual/index11.js

@@ -1,4 +1,4 @@
-var e = {};
+var e = { exports: {} };
 export {
-  e as __exports
+  e as __module
 };

package/dist/_virtual/index6.js

@@ -1,7 +1,5 @@
-import { getDefaultExportFromCjs as r } from "./_commonjsHelpers.js";
-import { __require as o } from "../node_modules/prop-types/index.js";
-var p = /* @__PURE__ */ o();
-const s = /* @__PURE__ */ r(p);
+import { __require as r } from "../node_modules/ics/dist/index.js";
+var i = r();
 export {
-  s as default
+  i as d
 };

package/dist/_virtual/index7.js

@@ -1,4 +1,7 @@
-var p = { exports: {} };
+import { getDefaultExportFromCjs as r } from "./_commonjsHelpers.js";
+import { __require as o } from "../node_modules/prop-types/index.js";
+var p = /* @__PURE__ */ o();
+const s = /* @__PURE__ */ r(p);
 export {
-  p as __module
+  s as default
 };

package/dist/_virtual/index8.js

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

package/dist/_virtual/index9.js

@@ -1,4 +1,4 @@
-var e = { exports: {} };
+var p = { exports: {} };
 export {
-  e as __module
+  p as __module
 };

package/dist/components/AddToCalendar/calendarLinks.js

@@ -1,4 +1,4 @@
-import { d as u } from "../../_virtual/index8.js";
+import { d as u } from "../../_virtual/index6.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/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/node_modules/ics/dist/index.js

@@ -1,4 +1,4 @@
-import { __exports as o } from "../../../_virtual/index10.js";
+import { __exports as o } from "../../../_virtual/index8.js";
 import { __require as q } from "./pipeline/index.js";
 var O;
 function R() {

package/dist/node_modules/ics/dist/pipeline/index.js

@@ -1,4 +1,4 @@
-import { __exports as u } from "../../../../_virtual/index11.js";
+import { __exports as u } from "../../../../_virtual/index10.js";
 import { __require as o } from "./build.js";
 import { __require as d } from "./format.js";
 import { __require as f } from "./validate.js";

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/index9.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/node_modules/prop-types/node_modules/react-is/index.js

@@ -1,4 +1,4 @@
-import { __module as e } from "../../../../_virtual/index9.js";
+import { __module as e } from "../../../../_virtual/index11.js";
 import { __require as o } from "./cjs/react-is.production.min.js";
 import { __require as t } from "./cjs/react-is.development.js";
 var r;

package/dist/node_modules/react-transition-group/esm/CSSTransition.js

@@ -1,7 +1,7 @@
 import C from "../../@babel/runtime/helpers/esm/extends.js";
 import x from "../../@babel/runtime/helpers/esm/objectWithoutPropertiesLoose.js";
 import N from "../../@babel/runtime/helpers/esm/inheritsLoose.js";
-import v from "../../../_virtual/index6.js";
+import v from "../../../_virtual/index7.js";
 import A from "../../dom-helpers/esm/addClass.js";
 import _ from "../../dom-helpers/esm/removeClass.js";
 import c from "react";

package/dist/node_modules/react-transition-group/esm/Transition.js

@@ -1,6 +1,6 @@
 import C from "../../@babel/runtime/helpers/esm/objectWithoutPropertiesLoose.js";
 import D from "../../@babel/runtime/helpers/esm/inheritsLoose.js";
-import o from "../../../_virtual/index6.js";
+import o from "../../../_virtual/index7.js";
 import v from "react";
 import b from "react-dom";
 import T from "./config.js";

package/dist/node_modules/react-transition-group/esm/utils/PropTypes.js

@@ -1,4 +1,4 @@
-import e from "../../../../_virtual/index6.js";
+import e from "../../../../_virtual/index7.js";
 var n = process.env.NODE_ENV !== "production" ? e.oneOfType([e.number, e.shape({
   enter: e.number,
   exit: e.number,

package/dist/scss/components/Button/_button.scss

@@ -1,5 +1,8 @@
+@use 'sass:color';
 @use '../../allPartials' as *;
 
+$destructive-hover-disabled: color.adjust($warning-default, $alpha: -0.5);
+
 .#{$px}-button {
   // we need this because we can't be sure that the Text component is being used as a child of the Button component
   @include text($labelMedium);
@@ -18,8 +21,10 @@
   padding: 0 $padding-sm;
   position: relative;
   transition:
-    color 0.25s,
-    background-color 0.25s;
+    color 0.25s ease-out,
+    background-color 0.25s ease-out,
+    border-color 0.25s ease-out,
+    box-shadow 0.25s ease-out;
 
   &--small {
     height: 32px; // Explicitly defined by design
@@ -50,20 +55,22 @@
 
     path {
       fill: $pure-white;
-      transition: 0.25s;
+      transition: fill 0.25s ease-out;
     }
   }
 
   &--primary,
   &--secondary,
-  &--tertiary {
+  &--tertiary,
+  &--destructive {
     &.#{$px}-link {
       text-decoration: none;
     }
   }
 
   &--primary.#{$px}-link,
-  &--secondary.#{$px}-link {
+  &--secondary.#{$px}-link,
+  &--destructive.#{$px}-link {
     font-variation-settings: 'wght' 600;
 
     .#{$px}-text {
@@ -168,6 +175,51 @@
     }
   }
 
+  &--destructive {
+    background-color: $white-100;
+    border: 2px solid $warning-default;
+    color: $warning-default;
+
+    &:focus-visible {
+      background-color: $white-100;
+      border-radius: $radius-3xl;
+      outline: 2px solid $warning-default;
+      outline-offset: 4px;
+    }
+
+    // Spinner inside destructive button uses the same red
+    .#{$px}-loader {
+      border-color: $warning-default;
+      border-bottom-color: transparent;
+    }
+
+    svg {
+      fill: $warning-default;
+
+      path {
+        fill: $warning-default;
+      }
+    }
+
+    @include hoverOrActive {
+      background-color: $white-100;
+      border-color: $destructive-hover-disabled;
+      color: $destructive-hover-disabled;
+
+      svg {
+        fill: $destructive-hover-disabled;
+
+        path {
+          fill: $destructive-hover-disabled;
+        }
+      }
+
+      &:disabled {
+        box-shadow: none;
+      }
+    }
+  }
+
   &--tertiary {
     background-color: transparent;
     color: $pure-black;
@@ -227,6 +279,20 @@
     border-color: transparent;
   }
 
+  &--destructive:disabled {
+    background-color: $white-100;
+    border-color: $destructive-hover-disabled;
+    color: $destructive-hover-disabled;
+
+    svg {
+      fill: $destructive-hover-disabled;
+
+      path {
+        fill: $destructive-hover-disabled;
+      }
+    }
+  }
+
   &--link:disabled {
     border: 0;
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@phillips/seldon",
-  "version": "1.240.1",
+  "version": "1.241.0",
   "repository": {
     "type": "git",
     "url": "https://github.com/PhillipsAuctionHouse/seldon"