From 3a4095b936cf9735fe72c744089b36f7300367ec Mon Sep 17 00:00:00 2001
From: Antoine Fricker <antoine@marmelab.com>
Date: Mon, 18 Jul 2022 14:38:38 +0200
Subject: [PATCH] Fix FF can not un-toggle filter through secondary action
 button

---
 .../src/list/filter/FilterListItem.tsx            | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/packages/ra-ui-materialui/src/list/filter/FilterListItem.tsx b/packages/ra-ui-materialui/src/list/filter/FilterListItem.tsx
index 994ed149409..482ab6cc877 100644
--- a/packages/ra-ui-materialui/src/list/filter/FilterListItem.tsx
+++ b/packages/ra-ui-materialui/src/list/filter/FilterListItem.tsx
@@ -189,14 +189,13 @@ export const FilterListItem = memo((props: FilterListItemProps) => {
                     data-selected={isSelected ? 'true' : 'false'}
                 />
                 {isSelected && (
-                    <ListItemSecondaryAction>
-                        <IconButton
-                            size="small"
-                            onClick={event => {
-                                event.stopPropagation();
-                                toggleFilter();
-                            }}
-                        >
+                    <ListItemSecondaryAction
+                        onClick={event => {
+                            event.stopPropagation();
+                            toggleFilter();
+                        }}
+                    >
+                        <IconButton size="small">
                             <CancelIcon />
                         </IconButton>
                     </ListItemSecondaryAction>