Merge pull request #10840 from google/fix/10428-warning-notices

Fix / 10428 Warning Notices

Author: tofumatt

assets/js/components/dashboard-sharing/DashboardSharingSettings/ModuleViewAccess.js

@@ -20,14 +20,15 @@
  * WordPress dependencies
  */
 import { __ } from '@wordpress/i18n';
-import { forwardRef } from '@wordpress/element';
+import { createInterpolateElement, forwardRef } from '@wordpress/element';
 
 /**
  * Internal dependencies
  */
 import PropTypes from 'prop-types';
 import UserRoleSelect from '../UserRoleSelect';
 import Notice from '../../Notice';
+import Link from '../../Link';
 
 const ModuleViewAccess = forwardRef(
 	(
@@ -53,17 +54,24 @@ const ModuleViewAccess = forwardRef(
 		if ( recoverable ) {
 			return (
 				<Notice
+					className="googlesitekit-notice--small"
 					type={ Notice.TYPES.WARNING }
-					description={ __(
-						'Managing user required to manage view access.',
-						'google-site-kit'
+					description={ createInterpolateElement(
+						__(
+							'Managing user required to manage view access. <a>Learn more</a>',
+							'google-site-kit'
+						),
+						{
+							a: (
+								<Link
+									href={ recoverableModuleSupportLink }
+									external
+									hideExternalIndicator
+								/>
+							),
+						}
 					) }
-					ctaButton={ {
-						label: 'Learn more',
-						href: recoverableModuleSupportLink,
-						external: true,
-						hideExternalIndicator: true,
-					} }
+					hideIcon
 				/>
 			);
 		}

assets/js/components/settings/EntityOwnershipChangeNotice.js

@@ -97,8 +97,9 @@ export default function EntityOwnershipChangeNotice( { slug } ) {
 
 	return (
 		<Notice
+			className="googlesitekit-notice--bottom-margin"
 			type={ Notice.TYPES.WARNING }
-			title={ sprintf(
+			description={ sprintf(
 				/* translators: %s: module name. */
 				__(
 					'By clicking confirm changes, you’re granting other users view-only access to data from %s via your Google account. You can always manage this later in the dashboard sharing settings.',