Merge pull request #7982 from marmelab/zd-691-FilterListItem-untoggle…

…-bug-firefox [Fix] Can't un-toggle filter through secondary action button in aside list
marmelab · Jul 18, 2022 · 07dd7ba · 07dd7ba
2 parents 05942f9 + 3a4095b
commit 07dd7ba
Showing 1 changed file with 7 additions and 8 deletions.
diff --git 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>