newspack-components 4.8.0 → 4.9.0-alpha.1

package/CHANGELOG.md

@@ -1,3 +1,13 @@
+# newspack-components [4.9.0-alpha.1](https://github.com/Automattic/newspack-workspace/compare/newspack-components@4.8.1-alpha.1...newspack-components@4.9.0-alpha.1) (2026-09-28)
+
+
+### Features
+
+* **audience:** refresh the Integrations brand marks ([#1155](https://github.com/Automattic/newspack-workspace/issues/1155)) ([af2729c](https://github.com/Automattic/newspack-workspace/commit/af2729cc6a5dd69cebe18456762eb21c55ae16a3))
+* **audience:** refresh the Integrations screen icons, order and copy ([#1151](https://github.com/Automattic/newspack-workspace/issues/1151)) ([1d3f5a6](https://github.com/Automattic/newspack-workspace/commit/1d3f5a6c074f18491b3047c0e679e6c4f7b1f3cd))
+* **reader-activation:** split Gravity Forms into its own integration ([#1170](https://github.com/Automattic/newspack-workspace/issues/1170)) ([7bd756b](https://github.com/Automattic/newspack-workspace/commit/7bd756bc30b12856091c126c6d054d45e6b78244))
+* **sync:** read the push log in a sync activity tab ([#1114](https://github.com/Automattic/newspack-workspace/issues/1114)) ([c1a2d2d](https://github.com/Automattic/newspack-workspace/commit/c1a2d2d299b8a0150ce1c4de37e14edf437176b3))
+
 # newspack-components [4.8.0](https://github.com/Automattic/newspack-workspace/compare/newspack-components@4.7.0...newspack-components@4.8.0) (2026-09-14)
 
 

package/DEVELOPMENT.md

@@ -100,7 +100,7 @@ When building a screen, use the **spacing scale** (8px unit: 16, 24, 32, 48, 64)
 
 - **`ActionCard`** – Use when one concept (e.g. a feature or setting) can be toggled on/off and may have extra content below. Internal padding (24px default; 16px/8px for isMedium/isSmall) and 24px between regions keep hierarchy clear; expandable content uses 24px top padding and 24px between siblings.
 - **`CollapsibleGroup`** / **`CollapsibleGroup.Item`** – A stack of independently collapsible items, separated by dividers and flush with the surrounding column. Not a W3C accordion: the items do not coordinate.
-- **`Notice`** – Use for outcome feedback (success/error/warning) or short contextual messages. Vertical margin is 32px so notices don’t collide with cards; keep one primary message per area when possible.
+- **`Notice`** – Being retired in favour of core’s `Notice` from `@wordpress/components`, which announces itself to assistive technology. Reach for core’s in new work; ours stays for the screens still on it. Where ours is used, vertical margin is 32px so notices don’t collide with cards, and one primary message per area reads best.
 - **`ProgressBar`** – Progress indicator.
 - **`StepsList`** / **`StepsListItem`** – Step-by-step list components.
 - **`StyleCard`** – Style preview card.
@@ -131,6 +131,7 @@ When building a screen, use the **spacing scale** (8px unit: 16, 24, 32, 48, 64)
 
 ### Utility Components
 
+- **`DebugBadge`** - Floating indicator that Newspack debug mode is on. Takes no props and renders nothing unless the site defines `WP_NEWSPACK_DEBUG`
 - **`GlobalNotices`** - Global notice system component
 - **`InfoButton`** - Reveals supplementary context from a `description` prop. Anything a reader needs in order to use a control belongs in visible help text instead
 - **`Modal`** - Modal dialog component
@@ -176,7 +177,7 @@ import {
 	ActionCard,
 	Button,
 	Card,
-	Notice,
+	SectionHeader,
 	TextControl,
 } from '../../../../../packages/components/src';
 ```
@@ -187,21 +188,21 @@ import {
 
 ```jsx
 // ✅ CORRECT - Within newspack-plugin monorepo
-import { Button, Card, Notice } from '../../../../../packages/components/src';
+import { Button, Card, SectionHeader } from '../../../../../packages/components/src';
 ```
 
 **As an npm package:** If `newspack-components` is installed as a dependency in another plugin, import from the package name:
 
 ```jsx
 // ✅ CORRECT - When installed as npm package
-import { Button, Card, Notice } from 'newspack-components';
+import { Button, Card, SectionHeader } from 'newspack-components';
 ```
 
 **Import individual components** – Import only what you need; do not import the whole namespace. List named imports **alphabetically** (e.g. from `@wordpress/components` or `newspack-components`):
 
 ```jsx
 // ✅ CORRECT – alphabetical
-import { Button, Card, Notice } from '../../../../../packages/components/src';
+import { Button, Card, SectionHeader } from '../../../../../packages/components/src';
 
 // ❌ WRONG – Don't import all
 import * as Components from '../../../../../packages/components/src';
@@ -253,7 +254,7 @@ This section shows how to use components **by context** (backend, blocks, fronte
 /**
  * WordPress dependencies
  */
-import { CheckboxControl, ExternalLink, RangeControl } from '@wordpress/components';
+import { CheckboxControl, ExternalLink, Notice, RangeControl } from '@wordpress/components';
 
 /**
  * Internal dependencies
@@ -262,9 +263,8 @@ import {
 	ActionCard,
 	Button,
 	Card,
-	Grid,
-	Notice,
 	Divider,
+	Grid,
 	SectionHeader,
 	TextControl,
 	Waiting,
@@ -273,14 +273,13 @@ import {
 
 **Example – Audience Setup Wizard (real reference):**
 ```jsx
-import { CheckboxControl, ExternalLink, RangeControl } from '@wordpress/components';
+import { CheckboxControl, ExternalLink, Notice, RangeControl } from '@wordpress/components';
 import {
 	ActionCard,
 	Button,
 	Card,
-	Grid,
-	Notice,
 	Divider,
+	Grid,
 	PluginInstaller,
 	SectionHeader,
 	TextControl,
@@ -291,7 +290,9 @@ import {
 export default withWizardScreen( ( { config, updateConfig } ) => {
 	return (
 		<>
-			<Notice noticeText={ __( 'Audience Management is enabled.', 'newspack-plugin' ) } isSuccess />
+			<Notice status="success" isDismissible={ false }>
+				{ __( 'Audience Management is enabled.', 'newspack-plugin' ) }
+			</Notice>
 			<Card noBorder>
 				<ActionCard
 					title={ __( 'Present newsletter signup after checkout', 'newspack-plugin' ) }
@@ -374,7 +375,7 @@ import { Fragment } from '@wordpress/element';
 /**
  * Internal dependencies
  */
-import { GlobalNotices, Notice, Wizard } from '../../../../../packages/components/src';
+import { GlobalNotices, Wizard } from '../../../../../packages/components/src';
 import sections from './sections';
 
 function Dashboard() {
@@ -541,7 +542,7 @@ When Newspack components don't provide what you need, use these WordPress compon
 ### Feedback and overlays
 
 - **`Spinner`** – Loading spinner
-- **`Notice`** – Inline notice (success/error/warning); prefer Newspack `Notice` in wizards when it fits
+- **`Notice`** – Inline notice (success/error/warning); prefer this over the Newspack `Notice`, which is being retired
 - **`Placeholder`** – Empty state in blocks
 - **`Modal`** – Modal dialog
 - **`Popover`** – Popover (e.g. webhooks endpoint actions, corrections modal)
@@ -622,7 +623,7 @@ Spacing is based on an **8px unit**. Use these values so new styles match existi
 | **8px** | Tight gaps, badge padding, small padding (e.g. ActionCard is-small) | Inline or dense UI |
 | **16px** | Gaps between related controls, buttons card gap, margins inside ActionCard region-children for Card/Grid/TextControl | Related items, form rows |
 | **24px** | Default ActionCard region padding, toggle/region gaps, expandable content padding and sibling spacing | Default internal padding and gaps within a card |
-| **32px** | Card vertical margin, Grid default gap and margin, SectionHeader first-child top, Notice margin, Divider margin | Section rhythm, between blocks |
+| **32px** | Card vertical margin, Grid default gap and margin, SectionHeader first-child top, Newspack Notice margin, Divider margin | Section rhythm, between blocks |
 | **48px** | SectionHeader container margin-top, Card horizontal padding (small screens) | Section separation |
 | **64px** | SectionHeader top margin, Divider margins (large breakpoint), buttons card margin, Card horizontal padding (large screens) | Major section separation |
 
@@ -693,7 +694,7 @@ Components rely on WordPress design system states where applicable; a few Newspa
 - **ActionCard (clickable)** – Hover: `box-shadow: 0 4px 8px rgba(black, 0.08)`; transition 125ms ease-in-out. Use for cards that navigate or open.
 - **Button** – Primary, secondary, disabled, and link variants follow `@wordpress/components` Button; focus and hover come from WordPress base styles.
 - **Toggle (inside ActionCard)** – Checked/unchecked and focus states from WordPress ToggleControl; label is visually hidden but available for accessibility.
-- **Notice** – Success (green), error (red), warning (yellow), info (gray) variants; use for feedback only and one primary message per area when possible.
+- **Notice** – Core's `Notice` carries the status in a tinted, bordered box (info, success, warning, error) and announces itself. The Newspack one, still in place on the screens that have not moved, tints the background and adds an icon. Either way, use notices for feedback only, and one primary message per area when possible.
 
 When adding new interactive components, preserve focus visibility and use the same state patterns (hover shadow, transition) so the UI feels consistent.
 
@@ -724,7 +725,7 @@ When adding new interactive components, preserve focus visibility and use the sa
 - **[@wordpress/components](https://www.npmjs.com/package/@wordpress/components)** – WordPress component library (fallback when Newspack components don’t exist)
 - **[@wordpress/base-styles](https://www.npmjs.com/package/@wordpress/base-styles)** – WordPress Design System styles and colors
 
-**Design resources:** For layout, components, and spacing, refer to the [WordPress Figma Library](https://www.figma.com/community/file/1149596986784498103/wordpress-design-library) and the [Block Editor Handbook](https://developer.wordpress.org/block-editor/reference-guides/components/). The **Components Demo** (`/wp-admin/admin.php?page=newspack-components-demo`) is the live reference for how Newspack components look and behave; use it to confirm spacing and hierarchy when building new screens.
+**Design resources:** For layout, components, and spacing, refer to the [WordPress Figma Library](https://www.figma.com/community/file/1149596986784498103/wordpress-design-library) and the [Block Editor Handbook](https://developer.wordpress.org/block-editor/reference-guides/components/). The **Components Demo** (`/wp-admin/admin.php?page=newspack-components-demo`) is the live reference for how Newspack components look and behave; use it to confirm spacing and hierarchy when building new screens. It no longer catalogues `Notice`; the Action Card examples are the place to see one rendered.
 
 ## Component Patterns
 

package/dist/cjs/action-card/index.js

@@ -9,6 +9,7 @@ var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/de
 var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
 var _components = require("@wordpress/components");
 var _element = require("@wordpress/element");
+var _htmlEntities = require("@wordpress/html-entities");
 var _i18n = require("@wordpress/i18n");
 var _icons = require("@wordpress/icons");
 var _ui = require("@wordpress/ui");
@@ -26,8 +27,80 @@ function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t =
  * Internal dependencies
  */ /**
  * External dependencies
- */ /**
+ */
+var NOTIFICATION_LEVELS = ['error', 'info', 'success', 'warning'];
+
+/**
+ * Collect a notification's text, including text nested inside React elements.
+ *
+ * Segments join with a space so that adjacent elements do not weld together: by the
+ * time speak() sees this string the markup that separated them is already gone.
+ *
+ * @param {*} value Notification content.
+ * @return {string} The collected text.
+ */
+var _getNotificationText = function getNotificationText(value) {
+  var _value$props;
+  if (typeof value === 'string') {
+    return value;
+  }
+  if (typeof value === 'number') {
+    return String(value);
+  }
+  if (value instanceof Error) {
+    return value.message;
+  }
+  if (Array.isArray(value)) {
+    return value.map(_getNotificationText).join(' ');
+  }
+  return (value === null || value === void 0 || (_value$props = value.props) === null || _value$props === void 0 ? void 0 : _value$props.children) === undefined ? '' : _getNotificationText(value.props.children);
+};
+
+/**
+ * Derive a plain-text announcement from a notification.
+ *
+ * Notice defaults spokenMessage to its children and runs renderToString over them
+ * during render, which corrupts the hook dispatcher when those children are
+ * components. A notification can be any element, so the announcement is always
+ * built from its text instead of letting that default apply. Tags in an HTML
+ * notification stay in, since speak() strips them itself. Entities are decoded so
+ * the announcement matches the text on screen, which getNotificationContent decodes
+ * for the same reason.
+ *
+ * @param {*} notification Notification content.
+ * @return {string} Message to announce.
+ */
+var getSpokenNotification = function getSpokenNotification(notification) {
+  return (0, _htmlEntities.decodeEntities)(_getNotificationText(notification)).replace(/\s+/g, ' ').trim();
+};
+
+/**
+ * Decode a notification's own text for display.
+ *
+ * React does not decode entities in text children, and notification text is often
+ * server-sourced, so a message carrying `&#8217;` would otherwise show the entity
+ * and announce the character. The HTML branch is left alone: there the browser's
+ * parser decodes as it renders.
+ *
+ * @param {*} value Notification content.
+ * @return {*} The content, with its own strings decoded.
+ */
+var _getNotificationContent = function getNotificationContent(value) {
+  if (value instanceof Error) {
+    return (0, _htmlEntities.decodeEntities)(value.message);
+  }
+  if (typeof value === 'string') {
+    return (0, _htmlEntities.decodeEntities)(value);
+  }
+  if (Array.isArray(value)) {
+    return value.map(_getNotificationContent);
+  }
+  return value;
+};
+
+/**
  * ActionCard component
+ *
  * @param {ActionCardProps} props Component props.
  * @return {JSX.Element} ActionCard component.
  */
@@ -124,6 +197,7 @@ var ActionCard = function ActionCard(_ref) {
     }
   }, [collapse]);
   var hasChildren = notification || children;
+  var notificationContent = _getNotificationContent(notification);
   var classes = (0, _classnames["default"])('newspack-action-card', simple && 'newspack-card--is-clickable', hasGreyHeader && 'newspack-card--has-grey-header', hasWhiteHeader && 'newspack-card--has-white-header', hasChildren && 'newspack-card--has-children', indent && 'newspack-card--indent', isSmall && 'is-small', isMedium && 'is-medium', checkbox && 'has-checkbox', expandable && 'is-expandable', draggable && 'is-draggable', actionContent && 'has-action-content', className);
   var backgroundImageStyles = function backgroundImageStyles(url) {
     return url ? {
@@ -247,24 +321,17 @@ var ActionCard = function ActionCard(_ref) {
           width: 24
         })
       })]
-    }), notification && /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
+    }), notification && NOTIFICATION_LEVELS.includes(notificationLevel) && /*#__PURE__*/(0, _jsxRuntime.jsx)("div", {
       className: "newspack-action-card__notification newspack-action-card__region-children",
-      children: ['error' === notificationLevel && /*#__PURE__*/(0, _jsxRuntime.jsx)(_.Notice, {
-        noticeText: notification,
-        isError: true,
-        rawHTML: notificationHTML
-      }), 'info' === notificationLevel && /*#__PURE__*/(0, _jsxRuntime.jsx)(_.Notice, {
-        noticeText: notification,
-        rawHTML: notificationHTML
-      }), 'success' === notificationLevel && /*#__PURE__*/(0, _jsxRuntime.jsx)(_.Notice, {
-        noticeText: notification,
-        isSuccess: true,
-        rawHTML: notificationHTML
-      }), 'warning' === notificationLevel && /*#__PURE__*/(0, _jsxRuntime.jsx)(_.Notice, {
-        noticeText: notification,
-        isWarning: true,
-        rawHTML: notificationHTML
-      })]
+      children: /*#__PURE__*/(0, _jsxRuntime.jsx)(_components.Notice, {
+        status: notificationLevel,
+        isDismissible: false,
+        spokenMessage: getSpokenNotification(notification),
+        children: notificationHTML && typeof notification === 'string' ? /*#__PURE__*/(0, _jsxRuntime.jsx)(_element.RawHTML, {
+          className: "newspack-action-card__notification-html",
+          children: notification
+        }) : notificationContent
+      })
     }), children && (expandable && expanded || !expandable) ? /*#__PURE__*/(0, _jsxRuntime.jsx)("div", {
       className: "newspack-action-card__region-children",
       children: children

package/dist/cjs/action-card/index.test.js

@@ -1,16 +1,26 @@
 "use strict";
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
+var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
+var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _react = require("@testing-library/react");
+var _a11y = require("@wordpress/a11y");
+var _element = require("@wordpress/element");
 var _index = _interopRequireDefault(require("./index"));
 var _jsxRuntime = require("react/jsx-runtime");
-/**
+function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
+function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2["default"])(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; } /**
  * External dependencies.
- */
-
-/**
+ */ /**
+ * WordPress dependencies.
+ */ /**
  * Internal dependencies.
  */
+jest.mock('@wordpress/a11y', function () {
+  return _objectSpread(_objectSpread({}, jest.requireActual('@wordpress/a11y')), {}, {
+    speak: jest.fn()
+  });
+});
 
 /**
  * The library Badge styles its wrapper rather than its text, so a badge with no label
@@ -18,7 +28,8 @@ var _jsxRuntime = require("react/jsx-runtime");
  * absent — a payment gateway reports no connection status until it is enabled, a prompt
  * can sit on a placement with no entry in the placement map — and an array literal is
  * always non-empty, so the component has to drop those itself.
- */describe('ActionCard badges', function () {
+ */
+describe('ActionCard badges', function () {
   var badgeText = function badgeText() {
     return _react.screen.queryAllByText(function (_, node) {
       return /__badge/.test((node === null || node === void 0 ? void 0 : node.className) || '');
@@ -88,4 +99,180 @@ var _jsxRuntime = require("react/jsx-runtime");
     // An empty array used to fall through as a literal `0` in the heading.
     expect(withEmptyArray.textContent).not.toContain('0');
   });
+});
+
+/**
+ * ActionCard derives the notice's spoken message rather than letting core default it to
+ * the children. Core runs `renderToString` over that default during render, which corrupts
+ * the hook dispatcher when the children are components — so the derivation is what keeps a
+ * notification carrying a `<Button>` from crashing the card on a later re-render.
+ */
+describe('ActionCard notifications', function () {
+  beforeEach(function () {
+    _a11y.speak.mockClear();
+  });
+  var spoken = function spoken() {
+    return _a11y.speak.mock.calls.map(function (call) {
+      return call[0];
+    });
+  };
+  it('renders a notice for a recognised level', function () {
+    var _render5 = (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_index["default"], {
+        title: "Ad Manager",
+        notification: "Plugin cannot be installed",
+        notificationLevel: "error"
+      })),
+      container = _render5.container;
+    expect(container.querySelector('.components-notice.is-error')).toBeInTheDocument();
+  });
+  it('renders no notice for an unrecognised or absent level', function () {
+    var _render6 = (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_index["default"], {
+        title: "Ad Manager",
+        notification: "Something happened",
+        notificationLevel: "critical"
+      })),
+      unknown = _render6.container;
+    expect(unknown.querySelector('.components-notice')).not.toBeInTheDocument();
+    var _render7 = (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_index["default"], {
+        title: "Ad Manager",
+        notification: "Something happened"
+      })),
+      absent = _render7.container;
+    expect(absent.querySelector('.components-notice')).not.toBeInTheDocument();
+  });
+  it('announces the text inside an element-only notification', function () {
+    (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_index["default"], {
+      title: "Ad Manager",
+      notificationLevel: "error",
+      notification: [/*#__PURE__*/(0, _jsxRuntime.jsx)("button", {
+        children: "Click here to connect your account."
+      }, "connect")]
+    }));
+    expect(spoken()).toContain('Click here to connect your account.');
+  });
+  it('announces strings and element text together', function () {
+    (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_index["default"], {
+      title: "Ad Manager",
+      notificationLevel: "success",
+      notification: ['Created custom targeting keys: ',
+      /*#__PURE__*/
+      // eslint-disable-next-line react/jsx-indent
+      (0, _jsxRuntime.jsx)("a", {
+        href: "https://example.org",
+        children: "Visit your GAM dashboard"
+      }, "dashboard")]
+    }));
+    expect(spoken()).toContain('Created custom targeting keys: Visit your GAM dashboard');
+  });
+  it('announces an Error and renders its message rather than throwing', function () {
+    (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_index["default"], {
+      title: "Ad Manager",
+      notificationLevel: "error",
+      notification: new Error('Network unreachable')
+    }));
+    expect(_react.screen.getByText('Network unreachable')).toBeInTheDocument();
+    expect(spoken()).toContain('Network unreachable');
+  });
+  it('renders HTML notifications in their own wrapper and hands speak() the tags to strip', function () {
+    var _render8 = (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_index["default"], {
+        title: "Ad Manager",
+        notificationLevel: "error",
+        notification: '<p>Install failed.</p><p>Try again.</p>',
+        notificationHTML: true
+      })),
+      container = _render8.container;
+    expect(container.querySelectorAll('.newspack-action-card__notification-html > p')).toHaveLength(2);
+    expect(spoken()[0]).toContain('<p>Install failed.</p>');
+  });
+  it('decodes entities in both the announcement and the visible text', function () {
+    (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_index["default"], {
+      title: "Ad Manager",
+      notificationLevel: "error",
+      notification: 'Plugin &#8220;Foo&#8221; could not be activated'
+    }));
+    expect(_react.screen.getByText('Plugin “Foo” could not be activated')).toBeInTheDocument();
+    expect(spoken()[0]).toContain('“Foo”');
+  });
+  it('decodes entities inside an array of notification parts', function () {
+    (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_index["default"], {
+      title: "Ad Manager",
+      notificationLevel: "error",
+      notification: ['Could not reach the publisher&#8217;s network.', ' ']
+    }));
+    expect(_react.screen.getByText(/Could not reach the publisher’s network\./)).toBeInTheDocument();
+    expect(spoken()[0]).toContain('publisher’s');
+  });
+  it('leaves entity decoding to the browser on the HTML branch', function () {
+    var _render9 = (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_index["default"], {
+        title: "Ad Manager",
+        notificationLevel: "error",
+        notification: 'Get it from <a href="https://example.org">the plugin&#8217;s site</a>',
+        notificationHTML: true
+      })),
+      container = _render9.container;
+    expect(container.querySelector('.newspack-action-card__notification-html a').textContent).toBe('the plugin’s site');
+    expect(spoken()[0]).toContain('plugin’s site');
+  });
+  it('renders a non-string notification as children even when notificationHTML is set', function () {
+    var _render0 = (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_index["default"], {
+        title: "Ad Manager",
+        notificationLevel: "error",
+        notification: /*#__PURE__*/(0, _jsxRuntime.jsx)("span", {
+          children: "Network unreachable"
+        }),
+        notificationHTML: true
+      })),
+      container = _render0.container;
+    expect(_react.screen.getByText('Network unreachable')).toBeInTheDocument();
+    expect(container.querySelector('.newspack-action-card__notification-html')).not.toBeInTheDocument();
+  });
+  it('keeps adjacent element texts apart in the announcement', function () {
+    (0, _react.render)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_index["default"], {
+      title: "Ad Manager",
+      notificationLevel: "error",
+      notification: [
+      /*#__PURE__*/
+      // eslint-disable-next-line react/jsx-indent
+      (0, _jsxRuntime.jsx)("a", {
+        href: "https://example.org",
+        children: "Visit your GAM dashboard"
+      }, "dashboard"), /*#__PURE__*/(0, _jsxRuntime.jsx)("button", {
+        type: "button",
+        children: "Click here to connect your account."
+      }, "connect")]
+    }));
+    expect(spoken()[0]).toBe('Visit your GAM dashboard Click here to connect your account.');
+  });
+  it('survives a re-render that changes how many hooked components the notification holds', function () {
+    var HookedAction = function HookedAction(_ref) {
+      var children = _ref.children;
+      var _useState = (0, _element.useState)(children),
+        _useState2 = (0, _slicedToArray2["default"])(_useState, 1),
+        label = _useState2[0];
+      return /*#__PURE__*/(0, _jsxRuntime.jsx)("button", {
+        type: "button",
+        children: label
+      });
+    };
+    var card = function card(notification) {
+      return /*#__PURE__*/(0, _jsxRuntime.jsx)(_index["default"], {
+        title: "Ad Manager",
+        notificationLevel: "error",
+        notification: notification
+      });
+    };
+    var _render1 = (0, _react.render)(card([/*#__PURE__*/(0, _jsxRuntime.jsx)(HookedAction, {
+        children: "Retry"
+      }, "retry"), /*#__PURE__*/(0, _jsxRuntime.jsx)(HookedAction, {
+        children: "Documentation"
+      }, "docs")])),
+      rerender = _render1.rerender,
+      container = _render1.container;
+    expect(function () {
+      return rerender(card([/*#__PURE__*/(0, _jsxRuntime.jsx)(HookedAction, {
+        children: "Retry"
+      }, "retry")]));
+    }).not.toThrow();
+    expect(container.querySelector('.components-notice.is-error')).toBeInTheDocument();
+  });
 });

package/dist/cjs/action-card/style.scss

@@ -339,18 +339,12 @@
 
 	/* Notifications */
 	.newspack-action-card__notification {
-		.newspack-notice {
+		.components-notice {
 			margin: 0;
-
-			&__heading {
-				margin-top: 0;
-			}
 		}
 
-		&.newspack-action-card__region-children {
-			.newspack-notice {
-				margin-top: 32px;
-			}
+		.newspack-action-card__notification-html > * {
+			margin: 0;
 		}
 	}
 

package/dist/cjs/card-feature/index.js

@@ -80,6 +80,30 @@ var _CardFeature = function CardFeature(_ref) {
   // translators: %1$s: the button's visible action label, e.g. "Enable". %2$s: the feature's name. The visible label must stay first (WCAG 2.5.3).
   (0, _i18n._x)('%1$s %2$s', 'accessible button name: visible action label, then feature name', 'newspack-plugin'), buttonLabel, title);
   var showMoreControls = enabled && !!(moreControls !== null && moreControls !== void 0 && moreControls.length) && (!requirements || requirementsActionable);
+  // A Configure button with nothing behind it would click to nowhere.
+  var showPrimaryButton = !isConfigureState || !!onConfigure;
+
+  // Enabling a feature with nothing to configure removes the button that was
+  // just used, so its focus goes to the More menu instead of the page. React
+  // detaches a ref before removing the node, which is the last point the
+  // button can still be read as focused.
+  var actionsRef = (0, _element.useRef)(null);
+  var primaryButtonNode = (0, _element.useRef)(null);
+  var primaryButtonLeftWithFocus = (0, _element.useRef)(false);
+  var primaryButtonRef = (0, _element.useCallback)(function (node) {
+    if (!node && primaryButtonNode.current) {
+      primaryButtonLeftWithFocus.current = primaryButtonNode.current === primaryButtonNode.current.ownerDocument.activeElement;
+    }
+    primaryButtonNode.current = node;
+  }, []);
+  (0, _element.useEffect)(function () {
+    var _actionsRef$current;
+    if (showPrimaryButton || !primaryButtonLeftWithFocus.current) {
+      return;
+    }
+    primaryButtonLeftWithFocus.current = false;
+    (_actionsRef$current = actionsRef.current) === null || _actionsRef$current === void 0 || (_actionsRef$current = _actionsRef$current.querySelector('.components-dropdown-menu__toggle')) === null || _actionsRef$current === void 0 || _actionsRef$current.focus();
+  }, [showPrimaryButton]);
   var handleButtonClick = function handleButtonClick() {
     if (isConfigureState) {
       onConfigure === null || onConfigure === void 0 || onConfigure();
@@ -89,8 +113,7 @@ var _CardFeature = function CardFeature(_ref) {
   };
   var iconDescriptor = icon && !(0, _element.isValidElement)(icon) ? icon : null;
   var iconClasses = iconDescriptor ? (0, _classnames["default"])('newspack-card-feature__icon', {
-    'newspack-card-feature__icon--radius-small': !!iconDescriptor.backgroundColor && iconDescriptor.radius !== 'full',
-    'newspack-card-feature__icon--radius-full': iconDescriptor.radius === 'full'
+    'newspack-card-feature__icon--has-background': !!iconDescriptor.backgroundColor
   }) : undefined;
   var renderedIcon = null;
   if ((0, _element.isValidElement)(icon)) {
@@ -137,10 +160,12 @@ var _CardFeature = function CardFeature(_ref) {
         gap: "sm",
         wrap: "wrap",
         children: [/*#__PURE__*/(0, _jsxRuntime.jsxs)(_ui.Stack, {
+          ref: actionsRef,
           direction: "row",
           align: "center",
           gap: "sm",
-          children: [/*#__PURE__*/(0, _jsxRuntime.jsx)(_button["default"], {
+          children: [showPrimaryButton && /*#__PURE__*/(0, _jsxRuntime.jsx)(_button["default"], {
+            ref: primaryButtonRef,
             variant: isConfigureState ? 'tertiary' : 'secondary',
             accessibleWhenDisabled: true,
             "aria-describedby": describedById,