From c90f1c5d1bb9fe376122774c9e807174ebe77b9d Mon Sep 17 00:00:00 2001
From: Hit Bhalodia <58802366+hbhalodia@users.noreply.github.com>
Date: Fri, 6 Sep 2024 21:08:44 +0530
Subject: [PATCH] Fix: Button: Replace remaining 40px default size violations
 [Block library 1] (#65033)

* Fix legacy comments to use 40px default button size

* Fix the embed placeholder button to use 40px default button size

* Fix freeform block to use 40px default button size

* Fix missing block to use 40px default button size

* Revert the instance for modal fullscreen button size

This is being reverted because we need to resolve this first - https://github.com/WordPress/gutenberg/pull/65033#discussion_r1746056089

Co-authored-by: hbhalodia <hbhalodia@git.wordpress.org>
Co-authored-by: mirka <0mirka00@git.wordpress.org>
---
 packages/block-library/src/comments/edit/comments-legacy.js | 3 +--
 packages/block-library/src/embed/embed-placeholder.js       | 6 ++----
 packages/block-library/src/freeform/modal.js                | 6 ++----
 packages/block-library/src/missing/edit.js                  | 3 +--
 4 files changed, 6 insertions(+), 12 deletions(-)

diff --git a/packages/block-library/src/comments/edit/comments-legacy.js b/packages/block-library/src/comments/edit/comments-legacy.js
index b5a12e34343da..dbe89a4e3a83e 100644
--- a/packages/block-library/src/comments/edit/comments-legacy.js
+++ b/packages/block-library/src/comments/edit/comments-legacy.js
@@ -29,8 +29,7 @@ export default function CommentsLegacy( {
 
 	const actions = [
 		<Button
-			// TODO: Switch to `true` (40px size) if possible
-			__next40pxDefaultSize={ false }
+			__next40pxDefaultSize
 			key="convert"
 			onClick={ () => void setAttributes( { legacy: false } ) }
 			variant="primary"
diff --git a/packages/block-library/src/embed/embed-placeholder.js b/packages/block-library/src/embed/embed-placeholder.js
index 9a26060a29412..1626aca5c9c9c 100644
--- a/packages/block-library/src/embed/embed-placeholder.js
+++ b/packages/block-library/src/embed/embed-placeholder.js
@@ -66,16 +66,14 @@ const EmbedPlaceholder = ( {
 						justify="flex-start"
 					>
 						<Button
-							// TODO: Switch to `true` (40px size) if possible
-							__next40pxDefaultSize={ false }
+							__next40pxDefaultSize
 							variant="secondary"
 							onClick={ tryAgain }
 						>
 							{ _x( 'Try again', 'button label' ) }
 						</Button>{ ' ' }
 						<Button
-							// TODO: Switch to `true` (40px size) if possible
-							__next40pxDefaultSize={ false }
+							__next40pxDefaultSize
 							variant="secondary"
 							onClick={ fallback }
 						>
diff --git a/packages/block-library/src/freeform/modal.js b/packages/block-library/src/freeform/modal.js
index 1768022377a43..aad79473b172b 100644
--- a/packages/block-library/src/freeform/modal.js
+++ b/packages/block-library/src/freeform/modal.js
@@ -123,8 +123,7 @@ export default function ModalEdit( props ) {
 					>
 						<FlexItem>
 							<Button
-								// TODO: Switch to `true` (40px size) if possible
-								__next40pxDefaultSize={ false }
+								__next40pxDefaultSize
 								variant="tertiary"
 								onClick={ onClose }
 							>
@@ -133,8 +132,7 @@ export default function ModalEdit( props ) {
 						</FlexItem>
 						<FlexItem>
 							<Button
-								// TODO: Switch to `true` (40px size) if possible
-								__next40pxDefaultSize={ false }
+								__next40pxDefaultSize
 								variant="primary"
 								onClick={ () => {
 									setAttributes( {
diff --git a/packages/block-library/src/missing/edit.js b/packages/block-library/src/missing/edit.js
index 70b032cb9a2fe..e7744839b0888 100644
--- a/packages/block-library/src/missing/edit.js
+++ b/packages/block-library/src/missing/edit.js
@@ -50,8 +50,7 @@ export default function MissingEdit( { attributes, clientId } ) {
 
 	const convertToHtmlButton = (
 		<Button
-			// TODO: Switch to `true` (40px size) if possible
-			__next40pxDefaultSize={ false }
+			__next40pxDefaultSize
 			key="convert"
 			onClick={ convertToHTML }
 			variant="primary"