@transferwise/components 46.28.0 → 46.29.1

package/src/logo/__snapshots__/Logo.spec.js.snap

@@ -8,7 +8,7 @@ exports[`Logo on mobile renders only fast flag 1`] = `
     role="img"
   >
     <svg
-      class="np-logo-svg"
+      class="np-logo-svg np-logo-svg--size-sm"
       fill="none"
       height="24"
       width="26"
@@ -19,6 +19,18 @@ exports[`Logo on mobile renders only fast flag 1`] = `
         fill="#163300"
       />
     </svg>
+    <svg
+      class="np-logo-svg np-logo-svg--size-md"
+      fill="none"
+      height="24"
+      width="106"
+      xmlns="http://www.w3.org/2000/svg"
+    >
+      <path
+        d="M58.738.359h6.498l-3.27 23.322h-6.498L58.739.359Zm-8.193 0L46.16 13.794 44.247.359h-4.545L33.96 13.754 33.243.36h-6.299l2.193 23.322h5.223l6.458-14.75 2.272 14.75h5.143L56.725.359h-6.18Zm54.558 13.555H89.674c.08 3.03 1.894 5.023 4.565 5.023 2.014 0 3.608-1.077 4.844-3.13l5.208 2.368C102.501 21.702 98.729 24 94.08 24c-6.339 0-10.545-4.266-10.545-11.123C83.535 5.342 88.478 0 95.455 0c6.14 0 10.007 4.146 10.007 10.605 0 1.076-.12 2.152-.359 3.309Zm-5.78-4.466c0-2.71-1.516-4.425-3.947-4.425-2.512 0-4.585 1.794-5.144 4.425h9.09ZM6.632 7.387 0 15.139h11.844l1.33-3.655H8.1l3.1-3.586.01-.095-2.016-3.471h9.072l-7.032 19.35h4.812L24.538.358H2.6l4.033 7.028Zm69.168-2.364c2.292 0 4.3 1.233 6.055 3.346l.921-6.575C81.143.688 78.93 0 76 0c-5.82 0-9.09 3.409-9.09 7.734 0 3 1.675 4.834 4.426 6.02l1.315.598c2.452 1.047 3.11 1.565 3.11 2.671 0 1.146-1.106 1.874-2.79 1.874-2.782.01-5.034-1.415-6.728-3.847l-.94 6.699C67.234 23.22 69.708 24 72.97 24c5.532 0 8.93-3.19 8.93-7.615 0-3.01-1.335-4.943-4.704-6.458l-1.435-.678c-1.994-.887-2.671-1.375-2.671-2.352 0-1.056.927-1.874 2.71-1.874Z"
+        fill="#163300"
+      />
+    </svg>
   </span>
 </div>
 `;
@@ -31,7 +43,19 @@ exports[`Logo renders wise business logo 1`] = `
     role="img"
   >
     <svg
-      class="np-logo-svg"
+      class="np-logo-svg np-logo-svg--size-sm"
+      fill="none"
+      height="24"
+      width="26"
+      xmlns="http://www.w3.org/2000/svg"
+    >
+      <path
+        d="M6.826 7.232 0 15.21h12.188l1.37-3.762H8.335l3.191-3.69.01-.098L9.461 4.09h9.336L11.56 24h4.953L25.25 0H2.676l4.15 7.232Z"
+        fill="#163300"
+      />
+    </svg>
+    <svg
+      class="np-logo-svg np-logo-svg--size-md"
       fill="none"
       height="24"
       width="106"
@@ -54,7 +78,19 @@ exports[`Logo renders wise business logo inversed 1`] = `
     role="img"
   >
     <svg
-      class="np-logo-svg"
+      class="np-logo-svg np-logo-svg--size-sm"
+      fill="none"
+      height="24"
+      width="26"
+      xmlns="http://www.w3.org/2000/svg"
+    >
+      <path
+        d="M6.826 7.232 0 15.21h12.188l1.37-3.762H8.335l3.191-3.69.01-.098L9.461 4.09h9.336L11.56 24h4.953L25.25 0H2.676l4.15 7.232Z"
+        fill="#9FE870"
+      />
+    </svg>
+    <svg
+      class="np-logo-svg np-logo-svg--size-md"
       fill="none"
       height="24"
       width="106"
@@ -77,7 +113,19 @@ exports[`Logo renders wise logo 1`] = `
     role="img"
   >
     <svg
-      class="np-logo-svg"
+      class="np-logo-svg np-logo-svg--size-sm"
+      fill="none"
+      height="24"
+      width="26"
+      xmlns="http://www.w3.org/2000/svg"
+    >
+      <path
+        d="M6.826 7.232 0 15.21h12.188l1.37-3.762H8.335l3.191-3.69.01-.098L9.461 4.09h9.336L11.56 24h4.953L25.25 0H2.676l4.15 7.232Z"
+        fill="#163300"
+      />
+    </svg>
+    <svg
+      class="np-logo-svg np-logo-svg--size-md"
       fill="none"
       height="24"
       width="106"
@@ -100,7 +148,19 @@ exports[`Logo renders wise logo by default 1`] = `
     role="img"
   >
     <svg
-      class="np-logo-svg"
+      class="np-logo-svg np-logo-svg--size-sm"
+      fill="none"
+      height="24"
+      width="26"
+      xmlns="http://www.w3.org/2000/svg"
+    >
+      <path
+        d="M6.826 7.232 0 15.21h12.188l1.37-3.762H8.335l3.191-3.69.01-.098L9.461 4.09h9.336L11.56 24h4.953L25.25 0H2.676l4.15 7.232Z"
+        fill="#163300"
+      />
+    </svg>
+    <svg
+      class="np-logo-svg np-logo-svg--size-md"
       fill="none"
       height="24"
       width="106"
@@ -123,7 +183,19 @@ exports[`Logo renders wise logo inversed 1`] = `
     role="img"
   >
     <svg
-      class="np-logo-svg"
+      class="np-logo-svg np-logo-svg--size-sm"
+      fill="none"
+      height="24"
+      width="26"
+      xmlns="http://www.w3.org/2000/svg"
+    >
+      <path
+        d="M6.826 7.232 0 15.21h12.188l1.37-3.762H8.335l3.191-3.69.01-.098L9.461 4.09h9.336L11.56 24h4.953L25.25 0H2.676l4.15 7.232Z"
+        fill="#9FE870"
+      />
+    </svg>
+    <svg
+      class="np-logo-svg np-logo-svg--size-md"
       fill="none"
       height="24"
       width="106"
@@ -146,7 +218,19 @@ exports[`Logo renders wise platform logo 1`] = `
     role="img"
   >
     <svg
-      class="np-logo-svg"
+      class="np-logo-svg np-logo-svg--size-sm"
+      fill="none"
+      height="18"
+      width="111"
+      xmlns="http://www.w3.org/2000/svg"
+    >
+      <path
+        d="M4.887 5.92 0 11.525h8.726l.98-2.642H5.969L8.252 6.29l.008-.068-1.486-2.508h6.683L8.276 17.7h3.546L18.079.84H1.916l2.97 5.08Zm17.114 11.8V.792h6.15c1.341 0 2.454.24 3.334.719.882.48 1.541 1.134 1.979 1.963.438.83.657 1.762.657 2.798s-.22 1.976-.662 2.806c-.44.829-1.104 1.485-1.988 1.967-.885.482-1.995.723-3.331.723h-4.23V9.603h3.994c.848 0 1.536-.144 2.064-.434.528-.29.914-.684 1.162-1.186.248-.501.37-1.07.37-1.71 0-.639-.122-1.206-.37-1.702-.248-.497-.636-.885-1.166-1.165-.531-.282-1.228-.422-2.093-.422h-3.268v14.735H22h.001ZM38.823.793v16.926h-2.518V.793h2.518ZM45.712 18c-.82 0-1.561-.15-2.224-.45a3.665 3.665 0 0 1-1.572-1.31c-.385-.573-.576-1.275-.576-2.107 0-.717.14-1.307.42-1.768.281-.462.661-.83 1.138-1.1a6.036 6.036 0 0 1 1.6-.612c.59-.139 1.19-.243 1.802-.314.775-.088 1.404-.16 1.887-.218.482-.058.834-.153 1.052-.286.22-.132.33-.347.33-.644v-.058c0-.72-.208-1.28-.62-1.678-.412-.396-1.025-.595-1.84-.595-.813 0-1.514.183-2 .55-.486.366-.822.772-1.007 1.218L41.736 8.1c.28-.772.692-1.395 1.234-1.873a5.034 5.034 0 0 1 1.874-1.041 7.562 7.562 0 0 1 2.232-.327c.516 0 1.065.06 1.647.178a4.949 4.949 0 0 1 1.634.65c.509.313.925.76 1.25 1.342.325.58.49 1.334.49 2.26v8.43h-2.46v-1.736h-.101c-.163.32-.408.634-.733.942-.326.309-.744.565-1.255.77-.512.204-1.123.305-1.836.305Zm.547-1.984c.696 0 1.294-.135 1.79-.405.497-.27.876-.623 1.138-1.062a2.71 2.71 0 0 0 .392-1.409v-1.637c-.09.089-.262.17-.518.244a7.313 7.313 0 0 1-.871.19c-.327.054-.644.1-.952.14l-.774.104c-.489.06-.933.162-1.336.306-.4.144-.722.348-.96.616-.239.267-.358.62-.358 1.062 0 .612.23 1.073.69 1.385.46.31 1.048.466 1.76.466ZM60.833 5.025V7.01h-7.067V5.025h7.067Zm-5.172-3.04h2.518v12.008c0 .48.074.838.22 1.078a1.1 1.1 0 0 0 .568.488c.233.086.487.128.763.128.201 0 .378-.014.53-.04l.354-.066.455 2.041a4.53 4.53 0 0 1-.624.17c-.27.058-.607.09-1.01.095a4.509 4.509 0 0 1-1.853-.347 3.166 3.166 0 0 1-1.39-1.124c-.355-.506-.531-1.143-.531-1.91V1.985Zm13.846 3.041V7.01h-7.312V5.026h7.312ZM64.2 17.72V3.555c0-.794.177-1.453.531-1.98.354-.525.823-.92 1.406-1.18a4.615 4.615 0 0 1 1.904-.393A6.072 6.072 0 0 1 70.14.34l-.59 2c-.117-.034-.27-.074-.454-.12a2.81 2.81 0 0 0-.674-.07c-.612 0-1.048.148-1.309.445-.26.297-.392.728-.392 1.29V17.72H64.2Zm12.182.257c-1.213 0-2.27-.273-3.175-.818-.904-.545-1.606-1.31-2.106-2.289-.5-.981-.75-2.127-.75-3.438 0-1.312.25-2.469.75-3.455.5-.986 1.202-1.753 2.106-2.298.904-.545 1.963-.818 3.175-.818 1.213 0 2.271.273 3.176.818.904.545 1.605 1.312 2.106 2.298.499.986.75 2.138.75 3.455 0 1.316-.251 2.457-.75 3.438-.5.981-1.202 1.744-2.106 2.289-.905.545-1.963.818-3.176.818Zm.008-2.075c.787 0 1.438-.203 1.954-.611.517-.41.9-.95 1.15-1.628.25-.678.374-1.425.374-2.24 0-.815-.124-1.555-.374-2.236-.25-.68-.633-1.227-1.15-1.64-.516-.415-1.167-.62-1.954-.62-.786 0-1.448.206-1.967.62-.519.412-.904.96-1.153 1.64a6.43 6.43 0 0 0-.375 2.236c0 .81.125 1.562.375 2.24.25.677.634 1.22 1.153 1.627.52.408 1.176.613 1.967.613Zm8.044 1.818V5.026h2.434v2.016h.135a3.098 3.098 0 0 1 1.25-1.617 3.618 3.618 0 0 1 2.034-.591 10.677 10.677 0 0 1 1.071.058v2.363a5.1 5.1 0 0 0-.538-.095 5.6 5.6 0 0 0-.775-.054c-.595 0-1.124.122-1.588.368-.464.245-.83.583-1.1 1.017-.27.432-.405.924-.405 1.475v7.752h-2.518v.002Zm8.589 0V5.025h2.418v2.067h.16c.27-.7.71-1.246 1.322-1.641.612-.394 1.345-.592 2.2-.592.854 0 1.59.198 2.177.596.587.396 1.021.942 1.302 1.637h.135c.309-.678.8-1.22 1.474-1.624.674-.405 1.477-.607 2.409-.607 1.174 0 2.132.361 2.876 1.082.744.722 1.117 1.81 1.117 3.265v8.512h-2.519V9.44c0-.86-.239-1.483-.716-1.869-.477-.386-1.047-.578-1.71-.578-.82 0-1.457.246-1.912.739-.455.492-.683 1.125-.683 1.896v8.092h-2.509V9.283c0-.69-.219-1.243-.657-1.662-.437-.419-1.007-.627-1.71-.627-.477 0-.917.122-1.318.368a2.74 2.74 0 0 0-.969 1.02c-.243.435-.365.94-.365 1.513v7.826h-2.519l-.003-.001Z"
+        fill="#163300"
+      />
+    </svg>
+    <svg
+      class="np-logo-svg np-logo-svg--size-md"
       fill="none"
       height="18"
       width="172"
@@ -169,7 +253,19 @@ exports[`Logo renders wise platform logo inversed 1`] = `
     role="img"
   >
     <svg
-      class="np-logo-svg"
+      class="np-logo-svg np-logo-svg--size-sm"
+      fill="none"
+      height="18"
+      width="111"
+      xmlns="http://www.w3.org/2000/svg"
+    >
+      <path
+        d="M4.887 5.918 0 11.523h8.726l.98-2.642H5.969l2.284-2.593.008-.068-1.486-2.508h6.683L8.276 17.699h3.546L18.079.838H1.916l2.97 5.08Zm17.114 11.799V.791h6.15c1.341 0 2.454.24 3.334.719.882.48 1.541 1.134 1.979 1.963.438.83.657 1.762.657 2.798s-.22 1.976-.662 2.806c-.44.83-1.104 1.485-1.988 1.967-.885.482-1.995.723-3.331.723h-4.23V9.6h3.994c.848 0 1.536-.144 2.064-.434.528-.29.914-.684 1.162-1.186.248-.501.37-1.07.37-1.71 0-.639-.122-1.206-.37-1.702-.248-.497-.636-.885-1.166-1.165-.531-.282-1.228-.422-2.093-.422h-3.268v14.735H22h.001ZM38.823.791v16.926h-2.518V.791h2.518Zm6.889 17.207c-.82 0-1.561-.15-2.224-.45a3.665 3.665 0 0 1-1.572-1.31c-.385-.573-.576-1.275-.576-2.107 0-.716.14-1.307.42-1.768.281-.462.661-.83 1.138-1.1a6.036 6.036 0 0 1 1.6-.612c.59-.139 1.19-.243 1.802-.313.775-.09 1.404-.161 1.887-.22.482-.057.834-.152 1.052-.285.22-.132.33-.347.33-.644v-.058c0-.72-.208-1.28-.62-1.678-.412-.396-1.025-.595-1.84-.595-.813 0-1.514.183-2 .55-.486.366-.822.772-1.007 1.218l-2.366-.528c.28-.772.692-1.395 1.234-1.872a5.034 5.034 0 0 1 1.874-1.042 7.562 7.562 0 0 1 2.232-.327c.516 0 1.065.06 1.647.178a4.949 4.949 0 0 1 1.634.65c.509.313.925.76 1.25 1.342.325.58.49 1.334.49 2.26v8.43h-2.46V15.98h-.101c-.163.32-.408.634-.733.942-.326.309-.744.565-1.255.77-.512.204-1.123.305-1.836.305Zm.547-1.984c.696 0 1.294-.135 1.79-.405.497-.27.876-.623 1.138-1.062a2.71 2.71 0 0 0 .392-1.409v-1.637c-.09.089-.262.17-.518.245a7.313 7.313 0 0 1-.871.19 33.88 33.88 0 0 1-.952.14l-.774.103c-.489.06-.933.162-1.336.306-.4.144-.722.348-.96.616-.239.267-.358.62-.358 1.062 0 .612.23 1.073.69 1.385.46.31 1.048.466 1.76.466ZM60.833 5.023v1.984h-7.067V5.023h7.067Zm-5.172-3.04h2.518V13.99c0 .48.074.838.22 1.078a1.1 1.1 0 0 0 .568.488c.233.086.487.128.763.128.201 0 .378-.014.53-.04l.354-.066.455 2.041a4.518 4.518 0 0 1-.624.17c-.27.058-.607.09-1.01.095a4.509 4.509 0 0 1-1.853-.347 3.165 3.165 0 0 1-1.39-1.124c-.355-.506-.531-1.143-.531-1.91V1.983Zm13.846 3.041v1.984h-7.312V5.024h7.312ZM64.2 17.718V3.553c0-.794.177-1.453.531-1.98.354-.525.823-.92 1.406-1.18A4.615 4.615 0 0 1 68.041 0a6.072 6.072 0 0 1 2.098.338l-.59 2c-.117-.034-.27-.074-.454-.12a2.81 2.81 0 0 0-.674-.07c-.612 0-1.048.148-1.309.445-.26.297-.392.728-.392 1.29v13.835H64.2Zm12.182.257c-1.213 0-2.27-.273-3.175-.818-.904-.545-1.606-1.31-2.106-2.289-.5-.981-.75-2.127-.75-3.438 0-1.312.25-2.469.75-3.455.5-.986 1.202-1.753 2.106-2.298.904-.545 1.963-.818 3.175-.818 1.213 0 2.271.273 3.176.818.904.545 1.605 1.312 2.106 2.298.499.986.75 2.138.75 3.455 0 1.316-.251 2.457-.75 3.438-.5.981-1.202 1.744-2.106 2.289-.905.545-1.963.818-3.176.818Zm.008-2.074c.787 0 1.438-.204 1.954-.612.517-.41.9-.95 1.15-1.628.25-.678.374-1.425.374-2.24 0-.815-.124-1.555-.374-2.236-.25-.68-.633-1.227-1.15-1.64-.516-.415-1.167-.62-1.954-.62-.786 0-1.448.206-1.967.62-.519.412-.904.96-1.153 1.64a6.43 6.43 0 0 0-.375 2.236c0 .81.125 1.562.375 2.24.25.677.634 1.22 1.153 1.628.52.407 1.176.612 1.967.612Zm8.044 1.817V5.024h2.434V7.04h.135a3.098 3.098 0 0 1 1.25-1.617 3.619 3.619 0 0 1 2.034-.591 10.666 10.666 0 0 1 1.071.058v2.363a5.1 5.1 0 0 0-.538-.095 5.6 5.6 0 0 0-.775-.054c-.595 0-1.124.122-1.588.368-.464.245-.83.584-1.1 1.017-.27.432-.405.924-.405 1.475v7.752h-2.518v.002Zm8.589 0V5.023h2.418V7.09h.16c.27-.7.71-1.246 1.322-1.641.612-.394 1.345-.592 2.2-.592.854 0 1.59.198 2.177.596.587.396 1.021.942 1.302 1.637h.135c.309-.678.8-1.22 1.474-1.624.674-.405 1.477-.607 2.409-.607 1.174 0 2.132.361 2.876 1.082.744.722 1.117 1.81 1.117 3.265v8.512h-2.519v-8.28c0-.86-.239-1.483-.716-1.869-.477-.386-1.047-.578-1.71-.578-.82 0-1.457.246-1.912.739-.455.492-.683 1.125-.683 1.896v8.092h-2.509V9.281c0-.69-.219-1.242-.657-1.662-.437-.419-1.007-.627-1.71-.627-.477 0-.917.122-1.318.368a2.74 2.74 0 0 0-.969 1.02c-.243.435-.365.94-.365 1.513v7.826h-2.522Z"
+        fill="#fff"
+      />
+    </svg>
+    <svg
+      class="np-logo-svg np-logo-svg--size-md"
       fill="none"
       height="18"
       width="172"

package/src/main.css

@@ -1857,11 +1857,6 @@ button.np-option {
   transition: opacity 150ms ease-in, height 150ms ease-in 150ms;
   opacity: 0;
 }
-.formatted-value__h3-custom-alignment {
-  margin-bottom: 16px;
-  margin-bottom: var(--size-16);
-  line-height: 24px;
-}
 .emphasis {
   font-weight: 600;
   font-weight: var(--font-weight-semi-bold);
@@ -3028,6 +3023,22 @@ a {
 .np-theme-personal--dark .np-logo-svg path {
   fill: var(--color-white);
 }
+.np-logo-svg--size-sm {
+  display: block;
+}
+@media (min-width: 576px) {
+  .np-logo-svg--size-sm {
+    display: none;
+  }
+}
+.np-logo-svg--size-md {
+  display: none;
+}
+@media (min-width: 576px) {
+  .np-logo-svg--size-md {
+    display: block;
+  }
+}
 .tw-modal--scrollable {
   max-height: 100%;
   /* mobile viewport bug fix */

package/src/main.less

@@ -22,7 +22,6 @@
 @import "./dimmer/Dimmer.less";
 @import "./drawer/Drawer.less";
 @import "./dropFade/DropFade.less";
-@import "./dynamicFieldDefinitionList/FormattedValue/FormattedValue.less";
 @import "./emphasis/Emphasis.less";
 @import "./flowNavigation/animatedLabel/AnimatedLabel.less";
 @import "./flowNavigation/backButton/BackButton.less";

package/src/moneyInput/MoneyInput.story.tsx

@@ -16,7 +16,7 @@ export default {
 
     return (
       <>
-        <label htmlFor={args.id as string}>Editable money input label</label>
+        <label htmlFor={args.id}>Editable money input label</label>
         <MoneyInput
           {...args}
           selectedCurrency={selectedCurrency}
@@ -160,13 +160,13 @@ export const SmallInput: Story = {
   render: (args) => {
     return (
       <>
-        <label htmlFor={args.id as string}>Money inputs</label>
+        <label htmlFor={args.id}>Money inputs</label>
         <MoneyInput {...args} {...SingleCurrency.args} />
         <br />
         <MoneyInput {...args} {...MultipleCurrencies.args} />
         <hr />
         <div className="has-error">
-          <label htmlFor={args.id as string}>Error states</label>
+          <label htmlFor={args.id}>Error states</label>
           <MoneyInput {...args} {...SingleCurrency.args} />
           <br />
           <MoneyInput {...args} {...MultipleCurrencies.args} />

package/src/nudge/Nudge.spec.tsx

@@ -51,14 +51,14 @@ describe('Nudge', () => {
     });
 
     it('does NOT render a nudge if localStorage has been set with the id of the nudge', () => {
-      jest.spyOn(window.localStorage.__proto__, 'getItem').mockReturnValue('["CAKE","TEST"]');
+      jest.spyOn(Storage.prototype, 'getItem').mockReturnValue('["CAKE","TEST"]');
 
       render(<Nudge {...defaultProps} id="TEST" persistDismissal />);
       expect(screen.queryByText('A nudge title')).not.toBeInTheDocument();
     });
 
     it('does NOT render a nudge if localStorage has been set with the id of the nudge AND calls is previously dismissed with TRUE', () => {
-      jest.spyOn(window.localStorage.__proto__, 'getItem').mockReturnValue('["CAKE","TEST"]');
+      jest.spyOn(Storage.prototype, 'getItem').mockReturnValue('["CAKE","TEST"]');
       const isPreviouslyDismissed = jest.fn();
 
       render(
@@ -75,7 +75,7 @@ describe('Nudge', () => {
     });
 
     it('shows a nudge if localStorage has been set with a different id for a different nudge and calls is previously dismissed with FALSE', () => {
-      jest.spyOn(window.localStorage.__proto__, 'getItem').mockReturnValue('["BANANA"]');
+      jest.spyOn(Storage.prototype, 'getItem').mockReturnValue('["BANANA"]');
       const isPreviouslyDismissed = jest.fn();
 
       render(
@@ -92,8 +92,8 @@ describe('Nudge', () => {
     });
 
     it('calls local storage with updated dismissed nudges value', () => {
-      jest.spyOn(window.localStorage.__proto__, 'getItem').mockReturnValue('["BANANA"]');
-      const setItem = jest.spyOn(window.localStorage.__proto__, 'setItem');
+      jest.spyOn(Storage.prototype, 'getItem').mockReturnValue('["BANANA"]');
+      const setItem = jest.spyOn(Storage.prototype, 'setItem');
 
       render(<Nudge {...defaultProps} id="TEST" persistDismissal />);
       expect(screen.getByText('A nudge title')).toBeInTheDocument();

package/src/phoneNumberInput/PhoneNumberInput.tsx

@@ -110,7 +110,8 @@ const PhoneNumberInput = ({
 
   useEffect(() => {
     if (broadcastedValue === null) {
-      return setBroadcastedValue(internalValue);
+      setBroadcastedValue(internalValue);
+      return;
     }
 
     const internalPhoneNumber = `${internalValue.prefix ?? ''}${internalValue.suffix}`;

package/src/phoneNumberInput/utils/cleanNumber/cleanNumber.ts

@@ -1,3 +1,3 @@
-const DIGITS_MATCH = /^$|^(\+)|([\d]+)/g;
+const DIGITS_MATCH = /^$|^(\+)|(\d+)/g;
 
 export const cleanNumber = (number: string) => number.match(DIGITS_MATCH)?.join('') ?? '';

package/src/popover/Popover.tsx

@@ -38,8 +38,9 @@ function resolvePlacement(preferredPlacement: PopoverPreferredPlacement) {
     case 'bottom-left':
     case 'bottom-right':
       return 'bottom';
+    default:
+      return preferredPlacement;
   }
-  return preferredPlacement;
 }
 
 export default function Popover({

package/src/promoCard/PromoCard.tsx

@@ -298,7 +298,7 @@ const PromoCard: FunctionComponent<PromoCardProps> = forwardRef(
             ...commonProps,
             'aria-checked':
               type === 'radio' ? value === state : type === 'checkbox' ? checked : undefined,
-            'aria-describedby': `${componentId}-title` || undefined,
+            'aria-describedby': `${componentId}-title`,
             'aria-disabled': isDisabled,
             'data-value': value ?? undefined,
             role: type === 'checkbox' || type === 'radio' ? type : undefined,

package/src/provider/theme/ThemeProvider.story.tsx

@@ -0,0 +1,21 @@
+import { Meta, StoryObj } from '@storybook/react';
+import { ThemeProvider } from '@wise/components-theming';
+
+import Button from '../../button';
+
+export default {
+  component: ThemeProvider,
+  title: 'Other/ThemeProvider',
+} satisfies Meta<typeof ThemeProvider>;
+
+type Story = StoryObj<typeof ThemeProvider>;
+
+export const PlatformTheme: Story = {
+  render: () => (
+    <>
+      <ThemeProvider theme="platform">
+        <Button>Button</Button>
+      </ThemeProvider>
+    </>
+  ),
+};

package/src/radioGroup/RadioGroup.spec.js

@@ -34,7 +34,7 @@ const RADIOS = [
 
 describe('RadioGroup', () => {
   it('renders null if no radios are provided', () => {
-    expect(shallow(<RadioGroup radios={[]} {...props} />).toBeNully);
+    expect(shallow(<RadioGroup radios={[]} {...props} />).isEmptyRender()).toBe(true);
   });
 
   it('renders radio options', () => {

package/src/section/Section.story.tsx

@@ -74,7 +74,8 @@ export const WithCards = () => {
 
   const handleOnCardClick = (index: number) => {
     if (expandedCardIndex === index) {
-      return setExpandedCardIndex(null);
+      setExpandedCardIndex(null);
+      return;
     }
     setExpandedCardIndex(index);
   };

package/src/segmentedControl/SegmentedControl.spec.tsx

@@ -1,5 +1,7 @@
 import '@testing-library/jest-dom';
-import { render, screen, userEvent } from '../test-utils';
+import React from 'react';
+
+import { render, screen, userEvent, waitFor } from '../test-utils';
 
 import SegmentedControl, { SegmentedControlProps } from './SegmentedControl';
 
@@ -30,7 +32,7 @@ const onChange = jest.fn();
 
 const defaultProps: SegmentedControlProps = {
   name: 'segmentedControl',
-  defaultValue: defaultSegments[0].value,
+  value: defaultSegments[0].value,
   mode: 'input',
   segments: defaultSegments,
   onChange,
@@ -86,6 +88,90 @@ describe('SegmentedControl', () => {
     expect(onChange).toHaveBeenCalledWith('reporting');
   });
 
+  it('does not call onChange on mount', async () => {
+    let onChangeCallCount = 0;
+
+    const ParentComponent = () => {
+      const [_, simulateRerender] = React.useState({});
+
+      // new function is created on every render
+      const onChange = () => {
+        onChangeCallCount += 1;
+        simulateRerender({});
+      };
+
+      return <SegmentedControl {...defaultProps} onChange={onChange} />;
+    };
+
+    render(<ParentComponent />);
+
+    await waitFor(() => {
+      expect(onChangeCallCount).toBe(0);
+    });
+  });
+
+  it('does not repeatedly call the onChange when the provided onChange prop changes on every render of its parent component', async () => {
+    let onChangeCallCount = 0;
+
+    const ParentComponent = () => {
+      const [_, simulateRerender] = React.useState({});
+
+      // a new onChange function is created on every render
+      const onChange = () => {
+        onChangeCallCount += 1;
+        simulateRerender({});
+      };
+
+      return <SegmentedControl {...defaultProps} onChange={onChange} />;
+    };
+
+    const { rerender } = render(<ParentComponent />);
+
+    rerender(<ParentComponent />);
+
+    await waitFor(() => {
+      expect(onChangeCallCount).toBe(0);
+    });
+  });
+
+  it('updates the selected segment when the selectedValue prop changes', () => {
+    const { rerender } = render(<SegmentedControl {...defaultProps} />);
+
+    const payroll = screen.getByRole('radio', { name: 'Payroll' });
+    userEvent.click(payroll);
+
+    expect(onChange).toHaveBeenCalledWith('payroll');
+
+    rerender(<SegmentedControl {...defaultProps} value="reporting" />);
+
+    const reporting = screen.getByRole('radio', { name: 'Reporting' });
+    expect(reporting).toBeChecked();
+  });
+
+  it('updates the options when the segments prop changes', () => {
+    const { rerender } = render(<SegmentedControl {...defaultProps} />);
+
+    const newSegments = [
+      {
+        id: '1',
+        value: 'payroll',
+        label: 'Payroll',
+      },
+      {
+        id: '3',
+        value: 'anotherOne',
+        label: 'Another One',
+      },
+    ];
+
+    rerender(<SegmentedControl {...defaultProps} segments={newSegments} />);
+
+    const anotherOne = screen.getByRole('radio', { name: 'Another One' });
+    userEvent.click(anotherOne);
+
+    expect(onChange).toHaveBeenCalledWith('anotherOne');
+  });
+
   it('throws error if user tries to add too many segments', () => {
     expect(() => {
       renderSegmentedControl({

package/src/segmentedControl/SegmentedControl.story.tsx

@@ -1,40 +1,78 @@
 import { StoryFn } from '@storybook/react';
 import React from 'react';
 
-import SegmentedControl, { Segments } from './SegmentedControl';
+import Button from '../button';
+
+import SegmentedControl from './SegmentedControl';
 
 export default {
   component: SegmentedControl,
   title: 'Forms/SegmentedControl',
 };
 
-const segments: Segments = [
-  { id: 'CUPCAKE', label: 'Cupcakes', value: 'cupcakes' },
-  { id: 'SPONGECAKE', label: 'Sponge cake', value: 'spongecake' },
-  { id: 'CARROT_CAKE', label: 'Carrot cake', value: 'carrotcake' },
-];
+const Template: StoryFn = (args) => {
+  const [segments, setSegments] = React.useState([
+    { id: 'CUPCAKE', label: 'Cupcakes', value: 'cupcakes' },
+    { id: 'SPONGECAKE', label: 'Sponge cake', value: 'spongecake' },
+    { id: 'CARROT_CAKE', label: 'Carrot cake', value: 'carrotcake' },
+  ]);
 
-const segmentsWithControls: Segments = [
-  { id: 'CUPCAKE', label: 'Cupcakes', value: 'cupcakes', controls: 'aControlId' },
-  { id: 'SPONGECAKE', label: 'Sponge cake', value: 'spongecake', controls: 'aControlId' },
-  { id: 'CARROT_CAKE', label: 'Carrot cake', value: 'carrotcake', controls: 'aControlId' },
-];
+  const [segmentsWithControls, setSegmentsWithControls] = React.useState([
+    { id: 'CUPCAKE', label: 'Cupcakes', value: 'cupcakes', controls: 'aControlId' },
+    { id: 'SPONGECAKE', label: 'Sponge cake', value: 'spongecake', controls: 'aControlId' },
+    { id: 'CARROT_CAKE', label: 'Carrot cake', value: 'carrotcake', controls: 'aControlId' },
+  ]);
 
-const Template: StoryFn = (args) => {
-  const [selectedValue, setSelectedValue] = React.useState(segments[0].value);
+  const [value, setValue] = React.useState(segments[0].value);
 
+  console.log('render: segments.length', segments.length);
   return (
     <div className="p-a-2">
       <SegmentedControl
         name="aSegmentedControl"
-        defaultValue={selectedValue}
-        onChange={setSelectedValue}
+        value={value}
+        onChange={setValue}
         {...(args.mode === 'view'
           ? { segments: segmentsWithControls, mode: 'view', controls: 'aControlId' }
           : { segments, mode: 'input' })}
       />
       <div className="m-a-2" id="aControlId">
-        <p>Selected value: {selectedValue}</p>
+        <p>Selected value: {value}</p>
+      </div>
+      <div className="m-a-2">
+        <p>
+          Force the <b>selectedValue</b> to be one of the following:
+          <ul>
+            {segments.map((segment) => (
+              <li key={segment.id}>
+                <a
+                  href="/"
+                  onClick={(e) => {
+                    e.preventDefault();
+                    setValue(segment.value);
+                  }}
+                >
+                  {segment.label}
+                </a>
+              </li>
+            ))}
+          </ul>
+        </p>
+      </div>
+      <div className="m-a-2">
+        <Button
+          priority="secondary"
+          type="danger"
+          size="sm"
+          disabled={segments.length < 2}
+          onClick={() => {
+            const index = segments.findIndex((s) => s.value !== value);
+            setSegments((prev) => prev.filter((_, i) => i !== index));
+            setSegmentsWithControls((prev) => prev.filter((_, i) => i !== index));
+          }}
+        >
+          Remove one segment
+        </Button>
       </div>
     </div>
   );