From ec1d6a44a4a27fb7c94627a8cedc50f2087f29d3 Mon Sep 17 00:00:00 2001
From: karthick-murugan <karthick.murugan02@gmail.com>
Date: Tue, 21 Jan 2025 10:38:33 +0530
Subject: [PATCH] Feedback updates

---
 .../block-library/src/latest-posts/edit.js    | 49 +++++++++++++------
 .../src/latest-posts/editor.scss              |  8 ---
 2 files changed, 34 insertions(+), 23 deletions(-)

diff --git a/packages/block-library/src/latest-posts/edit.js b/packages/block-library/src/latest-posts/edit.js
index b83993831f5ef2..3a334def3e52c1 100644
--- a/packages/block-library/src/latest-posts/edit.js
+++ b/packages/block-library/src/latest-posts/edit.js
@@ -52,6 +52,7 @@ import {
 	MAX_EXCERPT_LENGTH,
 	MAX_POSTS_COLUMNS,
 } from './constants';
+import { useToolsPanelDropdownMenuProps } from '../utils/hooks';
 
 /**
  * Module Constants
@@ -79,6 +80,8 @@ function getFeaturedImageDetails( post, size ) {
 
 export default function LatestPostsEdit( { attributes, setAttributes } ) {
 	const instanceId = useInstanceId( LatestPostsEdit );
+	const dropdownMenuProps = useToolsPanelDropdownMenuProps(); // Moved inside the component
+
 	const {
 		postsToShow,
 		order,
@@ -238,9 +241,10 @@ export default function LatestPostsEdit( { attributes, setAttributes } ) {
 						excerptLength: MIN_EXCERPT_LENGTH, // Default length for the excerpt
 					} )
 				}
+				dropdownMenuProps={ dropdownMenuProps }
 			>
 				<ToolsPanelItem
-					hasValue={ () => displayPostContent !== undefined }
+					hasValue={ () => !! displayPostContent }
 					label={ __( 'Post content' ) }
 					onDeselect={ () =>
 						setAttributes( { displayPostContent: false } )
@@ -253,13 +257,18 @@ export default function LatestPostsEdit( { attributes, setAttributes } ) {
 						onChange={ ( value ) =>
 							setAttributes( { displayPostContent: value } )
 						}
-						className={
-							displayPostContent
-								? 'latest-posts-toggle-control-margin-bottom'
-								: ''
-						}
 					/>
-					{ displayPostContent && (
+				</ToolsPanelItem>
+				{ displayPostContent && (
+					<ToolsPanelItem
+						hasValue={ () => displayPostContentRadio !== undefined }
+						label={ __( 'Show' ) }
+						onDeselect={ () =>
+							setAttributes( {
+								displayPostContentRadio: 'excerpt',
+							} )
+						}
+					>
 						<RadioControl
 							label={ __( 'Show' ) }
 							selected={ displayPostContentRadio }
@@ -276,9 +285,19 @@ export default function LatestPostsEdit( { attributes, setAttributes } ) {
 								} )
 							}
 						/>
-					) }
-					{ displayPostContent &&
-						displayPostContentRadio === 'excerpt' && (
+					</ToolsPanelItem>
+				) }
+				{ displayPostContent &&
+					displayPostContentRadio === 'excerpt' && (
+						<ToolsPanelItem
+							hasValue={ () => excerptLength !== undefined }
+							label={ __( 'Max number of words' ) }
+							onDeselect={ () =>
+								setAttributes( {
+									excerptLength: MIN_EXCERPT_LENGTH,
+								} )
+							}
+						>
 							<RangeControl
 								__nextHasNoMarginBottom
 								__next40pxDefaultSize
@@ -290,8 +309,8 @@ export default function LatestPostsEdit( { attributes, setAttributes } ) {
 								min={ MIN_EXCERPT_LENGTH }
 								max={ MAX_EXCERPT_LENGTH }
 							/>
-						) }
-				</ToolsPanelItem>
+						</ToolsPanelItem>
+					) }
 			</ToolsPanel>
 
 			<ToolsPanel
@@ -302,9 +321,10 @@ export default function LatestPostsEdit( { attributes, setAttributes } ) {
 						displayPostDate: false,
 					} )
 				}
+				dropdownMenuProps={ dropdownMenuProps }
 			>
 				<ToolsPanelItem
-					hasValue={ () => displayAuthor !== undefined }
+					hasValue={ () => !! displayAuthor }
 					label={ __( 'Display author name' ) }
 					onDeselect={ () =>
 						setAttributes( { displayAuthor: false } )
@@ -317,11 +337,10 @@ export default function LatestPostsEdit( { attributes, setAttributes } ) {
 						onChange={ ( value ) =>
 							setAttributes( { displayAuthor: value } )
 						}
-						className="latest-posts-toggle-control-margin-bottom"
 					/>
 				</ToolsPanelItem>
 				<ToolsPanelItem
-					hasValue={ () => displayPostDate !== undefined }
+					hasValue={ () => !! displayPostDate }
 					label={ __( 'Display post date' ) }
 					onDeselect={ () =>
 						setAttributes( { displayPostDate: false } )
diff --git a/packages/block-library/src/latest-posts/editor.scss b/packages/block-library/src/latest-posts/editor.scss
index 26ef7bcd55b5ff..fe901480974461 100644
--- a/packages/block-library/src/latest-posts/editor.scss
+++ b/packages/block-library/src/latest-posts/editor.scss
@@ -18,11 +18,3 @@
 		padding-left: 0;
 	}
 }
-
-.latest-posts-toggle-control-margin-bottom {
-	margin-bottom: 15px !important;
-}
-
-.latest-posts-toggle-control-margin-top {
-	margin-top: 15px !important;
-}