From 300968cca52e998ae2ca3aeb205b91836a573058 Mon Sep 17 00:00:00 2001
From: yogeshbhutkar <yogesh.bhutkar@rtcamp.com>
Date: Mon, 3 Feb 2025 15:23:14 +0530
Subject: [PATCH] refactor: hover on button to show `reset`

---
 .../components/global-styles/filters-panel.js | 90 +++++++++++--------
 .../src/components/global-styles/style.scss   |  7 +-
 2 files changed, 54 insertions(+), 43 deletions(-)

diff --git a/packages/block-editor/src/components/global-styles/filters-panel.js b/packages/block-editor/src/components/global-styles/filters-panel.js
index 5b42aedc324269..9099139729e122 100644
--- a/packages/block-editor/src/components/global-styles/filters-panel.js
+++ b/packages/block-editor/src/components/global-styles/filters-panel.js
@@ -10,7 +10,6 @@ import {
 	__experimentalToolsPanel as ToolsPanel,
 	__experimentalToolsPanelItem as ToolsPanelItem,
 	__experimentalItemGroup as ItemGroup,
-	__experimentalItem as Item,
 	__experimentalHStack as HStack,
 	__experimentalZStack as ZStack,
 	__experimentalDropdownContentWrapper as DropdownContentWrapper,
@@ -24,7 +23,7 @@ import {
 	Button,
 } from '@wordpress/components';
 import { __, _x } from '@wordpress/i18n';
-import { useCallback, useMemo } from '@wordpress/element';
+import { useCallback, useMemo, useRef } from '@wordpress/element';
 import { reset as resetIcon } from '@wordpress/icons';
 
 /**
@@ -122,35 +121,45 @@ const LabeledColorIndicator = ( { indicator, label } ) => (
 const renderToggle =
 	( duotone, resetDuotone ) =>
 	( { onToggle, isOpen } ) => {
+		const duotoneButtonRef = useRef( undefined );
+
 		const toggleProps = {
 			onClick: onToggle,
 			className: clsx( { 'is-open': isOpen } ),
 			'aria-expanded': isOpen,
+			ref: duotoneButtonRef,
+		};
+
+		const removeButtonProps = {
+			onClick: () => {
+				if ( isOpen ) {
+					onToggle();
+				}
+				resetDuotone();
+				// Return focus to parent button.
+				duotoneButtonRef.current?.focus();
+			},
+			className: 'block-editor-panel-duotone-settings__reset',
+			label: __( 'Remove' ),
 		};
 
 		return (
-			<ItemGroup
-				className="block-editor-panel-duotone-settings__dropdown"
-				isBordered
-				isSeparated
-			>
-				<Item as="button" { ...toggleProps }>
+			<>
+				<Button __next40pxDefaultSize { ...toggleProps }>
 					<LabeledColorIndicator
 						indicator={ duotone }
 						label={ __( 'Duotone' ) }
 					/>
-				</Item>
+				</Button>
 				{ duotone && (
 					<Button
 						__next40pxDefaultSize
-						label={ __( 'Reset' ) }
-						className="block-editor-panel-duotone-settings__reset"
 						size="small"
 						icon={ resetIcon }
-						onClick={ resetDuotone }
+						{ ...removeButtonProps }
 					/>
 				) }
-			</ItemGroup>
+			</>
 		);
 	};
 
@@ -216,31 +225,36 @@ export default function FiltersPanel( {
 					isShownByDefault={ defaultControls.duotone }
 					panelId={ panelId }
 				>
-					<Dropdown
-						popoverProps={ popoverProps }
-						className="block-editor-global-styles-filters-panel__dropdown"
-						renderToggle={ renderToggle( duotone, resetDuotone ) }
-						renderContent={ () => (
-							<DropdownContentWrapper paddingSize="small">
-								<MenuGroup label={ __( 'Duotone' ) }>
-									<p>
-										{ __(
-											'Create a two-tone color effect without losing your original image.'
-										) }
-									</p>
-									<DuotonePicker
-										colorPalette={ colorPalette }
-										duotonePalette={ duotonePalette }
-										// TODO: Re-enable both when custom colors are supported for block-level styles.
-										disableCustomColors
-										disableCustomDuotone
-										value={ duotone }
-										onChange={ setDuotone }
-									/>
-								</MenuGroup>
-							</DropdownContentWrapper>
-						) }
-					/>
+					<ItemGroup isBordered isSeparated>
+						<Dropdown
+							popoverProps={ popoverProps }
+							className="block-editor-global-styles-filters-panel__dropdown"
+							renderToggle={ renderToggle(
+								duotone,
+								resetDuotone
+							) }
+							renderContent={ () => (
+								<DropdownContentWrapper paddingSize="small">
+									<MenuGroup label={ __( 'Duotone' ) }>
+										<p>
+											{ __(
+												'Create a two-tone color effect without losing your original image.'
+											) }
+										</p>
+										<DuotonePicker
+											colorPalette={ colorPalette }
+											duotonePalette={ duotonePalette }
+											// TODO: Re-enable both when custom colors are supported for block-level styles.
+											disableCustomColors
+											disableCustomDuotone
+											value={ duotone }
+											onChange={ setDuotone }
+										/>
+									</MenuGroup>
+								</DropdownContentWrapper>
+							) }
+						/>
+					</ItemGroup>
 				</ToolsPanelItem>
 			) }
 		</Wrapper>
diff --git a/packages/block-editor/src/components/global-styles/style.scss b/packages/block-editor/src/components/global-styles/style.scss
index 1fd25815124f3d..4d778aab6b1d5b 100644
--- a/packages/block-editor/src/components/global-styles/style.scss
+++ b/packages/block-editor/src/components/global-styles/style.scss
@@ -24,6 +24,7 @@
 .block-editor-global-styles__shadow-dropdown {
 	display: block;
 	padding: 0;
+	position: relative;
 
 	button {
 		width: 100%;
@@ -78,10 +79,6 @@
 	direction: ltr;
 }
 
-.block-editor-panel-duotone-settings__dropdown {
-	position: relative;
-}
-
 .block-editor-panel-duotone-settings__reset {
 	position: absolute;
 	right: 0;
@@ -92,7 +89,7 @@
 		transition: opacity 0.1s ease-in-out;
 	}
 
-	.block-editor-panel-duotone-settings__dropdown:hover + &,
+	.block-editor-global-styles-filters-panel__dropdown:hover &,
 	&:focus,
 	&:hover {
 		opacity: 1;