newspack-components 4.8.0 → 4.9.0-alpha.1

package/dist/esm/card-feature/index.test.js

@@ -95,7 +95,8 @@ describe('CardFeature', function () {
     it('names the feature in the configure state too', function () {
       render(/*#__PURE__*/_jsx(CardFeature, {
         title: "Content Gifting",
-        enabled: true
+        enabled: true,
+        onConfigure: function onConfigure() {}
       }));
       expect(screen.getByRole('button', {
         name: 'Configure Content Gifting'
@@ -164,6 +165,72 @@ describe('CardFeature', function () {
       expect(onConfigure).toHaveBeenCalledTimes(1);
       expect(onEnable).not.toHaveBeenCalled();
     });
+
+    // A button reading Configure with nothing behind it would click to nowhere.
+    it('drops the primary button once enabled when there is nothing to configure', function () {
+      render(/*#__PURE__*/_jsx(CardFeature, {
+        title: "Content gifting",
+        enabled: true,
+        onEnable: jest.fn(),
+        moreControls: [{
+          title: 'Disable',
+          onClick: jest.fn()
+        }]
+      }));
+      expect(screen.queryByRole('button', {
+        name: /Configure/
+      })).toBeNull();
+      expect(moreMenu()).toBeInTheDocument();
+    });
+
+    // Enabling removes the button that was just used; without a hand-off,
+    // keyboard focus falls to the page.
+    it('hands focus to the More menu when enabling removes the focused button', function () {
+      var props = {
+        title: 'Content gifting',
+        onEnable: jest.fn(),
+        moreControls: [{
+          title: 'Disable',
+          onClick: jest.fn()
+        }]
+      };
+      var _render2 = render(/*#__PURE__*/_jsx(CardFeature, _objectSpread({}, props))),
+        rerender = _render2.rerender;
+      primaryButton().focus();
+      rerender(/*#__PURE__*/_jsx(CardFeature, _objectSpread(_objectSpread({}, props), {}, {
+        enabled: true
+      })));
+      expect(moreMenu()).toHaveFocus();
+    });
+    it('leaves focus alone when the removed button did not have it', function () {
+      var props = {
+        title: 'Content gifting',
+        onEnable: jest.fn(),
+        moreControls: [{
+          title: 'Disable',
+          onClick: jest.fn()
+        }]
+      };
+      var _render3 = render(/*#__PURE__*/_jsxs(_Fragment, {
+          children: [/*#__PURE__*/_jsx(CardFeature, _objectSpread({}, props)), /*#__PURE__*/_jsx("button", {
+            children: "Elsewhere"
+          })]
+        })),
+        rerender = _render3.rerender;
+      screen.getByRole('button', {
+        name: 'Elsewhere'
+      }).focus();
+      rerender(/*#__PURE__*/_jsxs(_Fragment, {
+        children: [/*#__PURE__*/_jsx(CardFeature, _objectSpread(_objectSpread({}, props), {}, {
+          enabled: true
+        })), /*#__PURE__*/_jsx("button", {
+          children: "Elsewhere"
+        })]
+      }));
+      expect(screen.getByRole('button', {
+        name: 'Elsewhere'
+      })).toHaveFocus();
+    });
     it('still routes to onEnable when enabled with an unmet requirement, since the button reads Enable', function () {
       var onEnable = jest.fn();
       var onConfigure = jest.fn();
@@ -221,18 +288,19 @@ describe('CardFeature', function () {
     it('leaves the enabled badge unlinked, since it explains nothing about the button', function () {
       render(/*#__PURE__*/_jsx(CardFeature, {
         title: "Content gifting",
-        enabled: true
+        enabled: true,
+        onConfigure: function onConfigure() {}
       }));
       expect(primaryButton()).not.toHaveAttribute('aria-describedby');
     });
     it('disables the button while an action is in flight, even with an actionable requirement', function () {
       var onEnable = jest.fn();
-      var _render2 = render(/*#__PURE__*/_jsx(CardFeature, {
+      var _render4 = render(/*#__PURE__*/_jsx(CardFeature, {
           title: "Content gifting",
           busy: true,
           onEnable: onEnable
         })),
-        rerender = _render2.rerender;
+        rerender = _render4.rerender;
       expect(primaryButton()).toHaveAttribute('aria-disabled', 'true');
       fireEvent.click(primaryButton());
       expect(onEnable).not.toHaveBeenCalled();
@@ -246,12 +314,12 @@ describe('CardFeature', function () {
       expect(primaryButton()).toHaveAttribute('aria-disabled', 'true');
     });
     it('accepts custom labels for both states', function () {
-      var _render3 = render(/*#__PURE__*/_jsx(CardFeature, {
+      var _render5 = render(/*#__PURE__*/_jsx(CardFeature, {
           title: "Apple News",
           enableLabel: "Connect",
           configureLabel: "Manage connection"
         })),
-        rerender = _render3.rerender;
+        rerender = _render5.rerender;
       expect(screen.getByRole('button', {
         name: 'Connect Apple News'
       })).toBeInTheDocument();
@@ -259,7 +327,8 @@ describe('CardFeature', function () {
         title: "Apple News",
         enabled: true,
         enableLabel: "Connect",
-        configureLabel: "Manage connection"
+        configureLabel: "Manage connection",
+        onConfigure: function onConfigure() {}
       }));
       expect(screen.getByRole('button', {
         name: 'Manage connection Apple News'
@@ -268,10 +337,10 @@ describe('CardFeature', function () {
   });
   describe('badge', function () {
     it('shows nothing when off, and the enabled badge when on', function () {
-      var _render4 = render(/*#__PURE__*/_jsx(CardFeature, {
+      var _render6 = render(/*#__PURE__*/_jsx(CardFeature, {
           title: "Content gifting"
         })),
-        rerender = _render4.rerender;
+        rerender = _render6.rerender;
       expect(screen.queryByText('Enabled')).not.toBeInTheDocument();
       rerender(/*#__PURE__*/_jsx(CardFeature, {
         title: "Content gifting",
@@ -306,11 +375,11 @@ describe('CardFeature', function () {
       onClick: jest.fn()
     }];
     it('appears only when enabled and controls are supplied', function () {
-      var _render5 = render(/*#__PURE__*/_jsx(CardFeature, {
+      var _render7 = render(/*#__PURE__*/_jsx(CardFeature, {
           title: "Content gifting",
           moreControls: controls
         })),
-        rerender = _render5.rerender;
+        rerender = _render7.rerender;
       expect(moreMenu()).not.toBeInTheDocument();
       rerender(/*#__PURE__*/_jsx(CardFeature, {
         title: "Content gifting",
@@ -325,14 +394,14 @@ describe('CardFeature', function () {
       expect(moreMenu()).toBeInTheDocument();
     });
     it('stays available on an actionable requirement and hides on a locked one', function () {
-      var _render6 = render(/*#__PURE__*/_jsx(CardFeature, {
+      var _render8 = render(/*#__PURE__*/_jsx(CardFeature, {
           title: "Content gifting",
           enabled: true,
           requirements: "Requires metering",
           requirementsActionable: true,
           moreControls: controls
         })),
-        rerender = _render6.rerender;
+        rerender = _render8.rerender;
       expect(moreMenu()).toBeInTheDocument();
       rerender(/*#__PURE__*/_jsx(CardFeature, {
         title: "Content gifting",
@@ -345,88 +414,74 @@ describe('CardFeature', function () {
   });
   describe('icon', function () {
     it('renders a ready element as-is, without the descriptor container', function () {
-      var _render7 = render(/*#__PURE__*/_jsx(CardFeature, {
+      var _render9 = render(/*#__PURE__*/_jsx(CardFeature, {
           title: "Content gifting",
           icon: /*#__PURE__*/_jsx("span", {
             "data-testid": "ready-icon"
           })
         })),
-        container = _render7.container;
+        container = _render9.container;
       expect(screen.getByTestId('ready-icon')).toBeInTheDocument();
       expect(container.querySelector('.newspack-card-feature__icon')).toBeNull();
     });
-    it('applies the descriptor colours inline and rounds fully on request', function () {
-      var _render8 = render(/*#__PURE__*/_jsx(CardFeature, {
+    it('applies the descriptor colours inline and rounds a backed icon', function () {
+      var _render0 = render(/*#__PURE__*/_jsx(CardFeature, {
           title: "Content gifting",
           icon: {
             node: /*#__PURE__*/_jsx("span", {
               "data-testid": "descriptor-icon"
             }),
             fill: '#003da5',
-            backgroundColor: '#dfe7f4',
-            radius: 'full'
+            backgroundColor: '#dfe7f4'
           }
         })),
-        container = _render8.container;
+        container = _render0.container;
       var iconContainer = container.querySelector('.newspack-card-feature__icon');
       expect(screen.getByTestId('descriptor-icon')).toBeInTheDocument();
-      expect(iconContainer).toHaveClass('newspack-card-feature__icon--radius-full');
+      expect(iconContainer).toHaveClass('newspack-card-feature__icon--has-background');
       expect(iconContainer).toHaveStyle({
         backgroundColor: '#dfe7f4',
         color: '#003da5'
       });
     });
     it('hides the descriptor container from assistive tech, since the title already names the feature', function () {
-      var _render9 = render(/*#__PURE__*/_jsx(CardFeature, {
+      var _render1 = render(/*#__PURE__*/_jsx(CardFeature, {
           title: "Content gifting",
           icon: {
             node: /*#__PURE__*/_jsx("svg", {})
           }
         })),
-        container = _render9.container;
+        container = _render1.container;
       expect(container.querySelector('.newspack-card-feature__icon')).toHaveAttribute('aria-hidden', 'true');
     });
-    it('falls back to small corners when a background is set without a radius', function () {
-      var _render0 = render(/*#__PURE__*/_jsx(CardFeature, {
-          title: "Content gifting",
-          icon: {
-            node: /*#__PURE__*/_jsx("span", {}),
-            backgroundColor: '#dfe7f4'
-          }
-        })),
-        container = _render0.container;
-      var iconContainer = container.querySelector('.newspack-card-feature__icon');
-      expect(iconContainer).toHaveClass('newspack-card-feature__icon--radius-small');
-      expect(iconContainer).not.toHaveClass('newspack-card-feature__icon--radius-full');
-    });
-    it('leaves an unbacked descriptor icon without a radius class', function () {
-      var _render1 = render(/*#__PURE__*/_jsx(CardFeature, {
+    it('leaves an unbacked descriptor icon without the background class', function () {
+      var _render10 = render(/*#__PURE__*/_jsx(CardFeature, {
           title: "Content gifting",
           icon: {
             node: /*#__PURE__*/_jsx("span", {}),
             fill: '#003da5'
           }
         })),
-        container = _render1.container;
+        container = _render10.container;
       var iconContainer = container.querySelector('.newspack-card-feature__icon');
-      expect(iconContainer).not.toHaveClass('newspack-card-feature__icon--radius-small');
+      expect(iconContainer).not.toHaveClass('newspack-card-feature__icon--has-background');
     });
   });
   it('passes className through to the card element', function () {
-    var _render10 = render(/*#__PURE__*/_jsx(CardFeature, {
+    var _render11 = render(/*#__PURE__*/_jsx(CardFeature, {
         title: "Content gifting",
         className: "newspack-subscribers__card"
       })),
-      container = _render10.container;
+      container = _render11.container;
     expect(container.querySelector('.newspack-card-feature')).toHaveClass('newspack-subscribers__card');
   });
   it('marks the card as muted only when requirements are set', function () {
-    var _render11 = render(/*#__PURE__*/_jsx(CardFeature, {
+    var _render12 = render(/*#__PURE__*/_jsx(CardFeature, {
         title: "Content gifting",
         enabled: true
       })),
-      container = _render11.container,
-      rerender = _render11.rerender;
+      container = _render12.container,
+      rerender = _render12.rerender;
     expect(container.querySelector('.newspack-card-feature--muted')).toBeNull();
     rerender(/*#__PURE__*/_jsx(CardFeature, {
       title: "Content gifting",

package/dist/esm/card-feature/style.scss

@@ -49,11 +49,7 @@
 			width: wp-vars.$grid-unit-30;
 		}
 
-		&--radius-small {
-			border-radius: wp-vars.$radius-small;
-		}
-
-		&--radius-full {
+		&--has-background {
 			border-radius: wp-vars.$radius-round;
 		}
 	}

package/dist/esm/debug-badge/index.js

@@ -0,0 +1,39 @@
+/**
+ * Debug Badge
+ */
+
+/**
+ * WordPress dependencies.
+ */
+import { __ } from '@wordpress/i18n';
+import { Icon, bug } from '@wordpress/icons';
+
+/**
+ * Internal dependencies.
+ */
+import './style.scss';
+
+/**
+ * Debug Badge component.
+ *
+ * Gates itself on `window.newspack_aux_data.is_debug_mode`, so consumers render
+ * it unconditionally and it stays absent outside debug mode.
+ *
+ * @return {JSX.Element|null} Debug Badge component, or nothing outside debug mode.
+ */
+import { jsx as _jsx } from "react/jsx-runtime";
+var DebugBadge = function DebugBadge() {
+  var _window$newspack_aux_;
+  if (!((_window$newspack_aux_ = window.newspack_aux_data) !== null && _window$newspack_aux_ !== void 0 && _window$newspack_aux_.is_debug_mode)) {
+    return null;
+  }
+  return /*#__PURE__*/_jsx("div", {
+    className: "newspack-debug-badge",
+    role: "img",
+    "aria-label": __('Debug mode', 'newspack-plugin'),
+    children: /*#__PURE__*/_jsx(Icon, {
+      icon: bug
+    })
+  });
+};
+export default DebugBadge;

package/dist/esm/debug-badge/index.test.js

@@ -0,0 +1,46 @@
+/**
+ * External dependencies
+ */
+import { render, screen } from '@testing-library/react';
+
+/**
+ * Internal dependencies
+ */
+import DebugBadge from '.';
+import { jsx as _jsx } from "react/jsx-runtime";
+describe('DebugBadge', function () {
+  afterEach(function () {
+    delete window.newspack_aux_data;
+  });
+  it('renders the badge and its glyph in debug mode', function () {
+    window.newspack_aux_data = {
+      is_debug_mode: true
+    };
+    var _render = render(/*#__PURE__*/_jsx(DebugBadge, {})),
+      container = _render.container;
+    expect(container.querySelector('.newspack-debug-badge')).toBeInTheDocument();
+    expect(container.querySelector('.newspack-debug-badge > svg')).toBeInTheDocument();
+  });
+  it('names itself for assistive tech', function () {
+    window.newspack_aux_data = {
+      is_debug_mode: true
+    };
+    render(/*#__PURE__*/_jsx(DebugBadge, {}));
+    expect(screen.getByRole('img', {
+      name: 'Debug mode'
+    })).toBeInTheDocument();
+  });
+  it('renders nothing outside debug mode', function () {
+    window.newspack_aux_data = {
+      is_debug_mode: false
+    };
+    var _render2 = render(/*#__PURE__*/_jsx(DebugBadge, {})),
+      container = _render2.container;
+    expect(container).toBeEmptyDOMElement();
+  });
+  it('renders nothing when the global is absent', function () {
+    var _render3 = render(/*#__PURE__*/_jsx(DebugBadge, {})),
+      container = _render3.container;
+    expect(container).toBeEmptyDOMElement();
+  });
+});

package/dist/esm/debug-badge/style.scss

@@ -0,0 +1,26 @@
+/**
+ * Debug Badge
+ */
+
+@use "~@wordpress/base-styles/colors" as wp-colors;
+@use "~@wordpress/base-styles/variables" as wp-vars;
+@use "../../../colors/colors.module" as np-colors;
+
+.newspack-debug-badge {
+	background: np-colors.$primary-600;
+	border-radius: wp-vars.$radius-round;
+	bottom: 0;
+	box-shadow: var(--wpds-elevation-sm, #{wp-vars.$elevation-small});
+	display: grid;
+	height: wp-vars.$button-size;
+	margin: 0 wp-vars.$grid-unit-30 wp-vars.$grid-unit-30;
+	place-items: center;
+	pointer-events: none;
+	position: fixed;
+	width: wp-vars.$button-size;
+	z-index: 9997;
+
+	> svg {
+		fill: wp-colors.$white;
+	}
+}

package/dist/esm/empty-state/index.test.js

@@ -49,17 +49,30 @@ describe('EmptyState.Root', function () {
       gap: 'var(--wpds-dimension-gap-2xl, 32px)'
     });
   });
+  it('tightens the stack gap when small', function () {
+    var _render4 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
+        size: "small",
+        children: /*#__PURE__*/_jsx("p", {
+          children: "body"
+        })
+      })),
+      container = _render4.container;
+    var stack = container.querySelector('.newspack-empty-state').firstElementChild;
+    expect(stack).toHaveStyle({
+      gap: 'var(--wpds-dimension-gap-xl, 24px)'
+    });
+  });
 
   // Consumers key `:has()` selectors off this class, so losing it changes their
   // layout without failing anything.
   it('merges className onto the grid', function () {
-    var _render4 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
+    var _render5 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
         className: "consumer-empty-state",
         children: /*#__PURE__*/_jsx("p", {
           children: "body"
         })
       })),
-      container = _render4.container;
+      container = _render5.container;
     expect(container.querySelector('.newspack-empty-state')).toHaveClass('consumer-empty-state');
   });
 
@@ -76,13 +89,13 @@ describe('EmptyState.Root', function () {
 });
 describe('EmptyState.Header', function () {
   it('renders the title and description in the header', function () {
-    var _render5 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
+    var _render6 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
         children: /*#__PURE__*/_jsx(EmptyState.Header, {
           title: "Get started with newsletters",
           description: "Compose and send."
         })
       })),
-      container = _render5.container;
+      container = _render6.container;
     expect(container.querySelector('.newspack-empty-state__title')).toHaveTextContent('Get started with newsletters');
     expect(container.querySelector('.newspack-empty-state__description')).toHaveTextContent('Compose and send.');
   });
@@ -90,13 +103,13 @@ describe('EmptyState.Header', function () {
   // Spacing comes from the stacks, so a margin creeping back onto the heading or
   // paragraph would silently widen every gap.
   it('spaces the header from its stacks rather than from margins', function () {
-    var _render6 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
+    var _render7 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
         children: /*#__PURE__*/_jsx(EmptyState.Header, {
           title: "Get started with newsletters",
           description: "Compose and send."
         })
       })),
-      container = _render6.container;
+      container = _render7.container;
     var header = container.querySelector('.newspack-empty-state__header');
     expect(header).toHaveStyle({
       flexDirection: 'column',
@@ -110,33 +123,33 @@ describe('EmptyState.Header', function () {
     });
   });
   it('wraps the icon in the disc', function () {
-    var _render7 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
+    var _render8 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
         children: /*#__PURE__*/_jsx(EmptyState.Header, {
           title: "Get started with newsletters",
           icon: /*#__PURE__*/_jsx("svg", {})
         })
       })),
-      container = _render7.container;
+      container = _render8.container;
     expect(container.querySelector('.newspack-empty-state__icon svg')).toBeInTheDocument();
   });
   it('omits the description and the icon when they are not given', function () {
-    var _render8 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
+    var _render9 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
         children: /*#__PURE__*/_jsx(EmptyState.Header, {
           title: "Get started with newsletters"
         })
       })),
-      container = _render8.container;
+      container = _render9.container;
     expect(container.querySelector('.newspack-empty-state__description')).not.toBeInTheDocument();
     expect(container.querySelector('.newspack-empty-state__icon')).not.toBeInTheDocument();
   });
   it('renders an h2 and a 48px icon at the default size', function () {
-    var _render9 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
+    var _render0 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
         children: /*#__PURE__*/_jsx(EmptyState.Header, {
           title: "Get started with newsletters",
           icon: /*#__PURE__*/_jsx("svg", {})
         })
       })),
-      container = _render9.container;
+      container = _render0.container;
     expect(screen.getByRole('heading', {
       level: 2,
       name: 'Get started with newsletters'
@@ -145,14 +158,14 @@ describe('EmptyState.Header', function () {
     expect(container.querySelector('.newspack-empty-state__icon svg')).toHaveAttribute('width', '48');
   });
   it('drops to an h3, the small modifier and a 24px icon when the root is small', function () {
-    var _render0 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
+    var _render1 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
         size: "small",
         children: /*#__PURE__*/_jsx(EmptyState.Header, {
           title: "No products match this rule",
           icon: /*#__PURE__*/_jsx("svg", {})
         })
       })),
-      container = _render0.container;
+      container = _render1.container;
     expect(container.querySelector('.newspack-empty-state__header--small')).toBeInTheDocument();
     expect(screen.getByRole('heading', {
       level: 3,
@@ -177,13 +190,13 @@ describe('EmptyState.Header', function () {
     })).toBeInTheDocument();
   });
   it('carries its own class hook alongside any className', function () {
-    var _render1 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
+    var _render10 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
         children: /*#__PURE__*/_jsx(EmptyState.Header, {
           title: "Get started with newsletters",
           className: "consumer-header"
         })
       })),
-      container = _render1.container;
+      container = _render10.container;
     var header = container.querySelector('.newspack-empty-state__header');
     expect(header).toBeInTheDocument();
     expect(header).toHaveClass('consumer-header');
@@ -207,11 +220,11 @@ describe('EmptyState.Header', function () {
     var previous = process.env.NODE_ENV;
     process.env.NODE_ENV = 'production';
     try {
-      var _render10 = render(/*#__PURE__*/_jsx(EmptyState.Header, {
+      var _render11 = render(/*#__PURE__*/_jsx(EmptyState.Header, {
           title: "Orphan",
           icon: /*#__PURE__*/_jsx("svg", {})
         })),
-        container = _render10.container;
+        container = _render11.container;
       expect(screen.getByRole('heading', {
         level: 2,
         name: 'Orphan'
@@ -225,7 +238,7 @@ describe('EmptyState.Header', function () {
 });
 describe('EmptyState.Actions', function () {
   it('renders its children in a centred row', function () {
-    var _render11 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
+    var _render12 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
         children: /*#__PURE__*/_jsx(EmptyState.Actions, {
           children: /*#__PURE__*/_jsx("button", {
             type: "button",
@@ -233,7 +246,7 @@ describe('EmptyState.Actions', function () {
           })
         })
       })),
-      container = _render11.container;
+      container = _render12.container;
     expect(container.querySelector('.newspack-empty-state__actions')).toBeInTheDocument();
     expect(container.querySelector('.newspack-empty-state__actions')).toHaveStyle({
       justifyContent: 'center',
@@ -244,7 +257,7 @@ describe('EmptyState.Actions', function () {
     })).toBeInTheDocument();
   });
   it('stacks into a column while keeping the hook class', function () {
-    var _render12 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
+    var _render13 = render(/*#__PURE__*/_jsx(EmptyState.Root, {
         children: /*#__PURE__*/_jsxs(EmptyState.Actions, {
           orientation: "column",
           children: [/*#__PURE__*/_jsx("button", {
@@ -256,7 +269,7 @@ describe('EmptyState.Actions', function () {
           })]
         })
       })),
-      container = _render12.container;
+      container = _render13.container;
     var actions = container.querySelector('.newspack-empty-state__actions');
     expect(actions).toBeInTheDocument();
     expect(actions).toHaveStyle({
@@ -287,10 +300,10 @@ describe('EmptyState.Actions', function () {
     var previous = process.env.NODE_ENV;
     process.env.NODE_ENV = 'production';
     try {
-      var _render13 = render(/*#__PURE__*/_jsx(EmptyState.Actions, {
+      var _render14 = render(/*#__PURE__*/_jsx(EmptyState.Actions, {
           children: "x"
         })),
-        container = _render13.container;
+        container = _render14.container;
       expect(container.querySelector('.newspack-empty-state__actions')).toBeInTheDocument();
     } finally {
       process.env.NODE_ENV = previous;

package/dist/esm/empty-state/root.js

@@ -37,6 +37,7 @@ var Root = function Root(_ref) {
       size: size
     };
   }, [size]);
+  var isSmall = size === 'small';
   return /*#__PURE__*/_jsx(EmptyStateContext.Provider, {
     value: context,
     children: /*#__PURE__*/_jsx(Grid, {
@@ -46,7 +47,7 @@ var Root = function Root(_ref) {
       children: /*#__PURE__*/_jsx(Stack, _objectSpread(_objectSpread({
         className: "newspack-empty-state__stack",
         direction: "column",
-        gap: "2xl"
+        gap: isSmall ? 'xl' : '2xl'
       }, gridColumn), {}, {
         children: children
       }))