Merge pull request #3956 from ProjectMirador/owner-state-destructure

Simplify the parameters passed to the button
ProjectMirador · Oct 29, 2024 · 6a0d2a4 · 6a0d2a4
2 parents 74283f5 + bd11b94
commit 6a0d2a4
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/components/MiradorMenuButton.js b/src/components/MiradorMenuButton.js
@@ -4,9 +4,9 @@ import Badge from '@mui/material/Badge';
 import IconButton from '@mui/material/IconButton';
 import Tooltip from '@mui/material/Tooltip';
 
-const Root = styled(IconButton, { name: 'MiradorMenuButton', slot: 'root' })(({ ownerState, theme }) => ({
+const Root = styled(IconButton, { name: 'MiradorMenuButton', slot: 'root' })(({ selected, theme }) => ({
   fill: 'currentcolor',
-  ...(ownerState.selected && {
+  ...(selected && {
     backgroundColor: theme.palette.action.selected,
   }),
 }));
@@ -23,6 +23,7 @@ export function MiradorMenuButton(props) {
     children,
     container,
     dispatch,
+    selected,
     BadgeProps,
     TooltipProps,
     sx,
@@ -31,7 +32,7 @@ export function MiradorMenuButton(props) {
 
   const button = (
     <Root
-      ownerState={props}
+      selected={selected}
       {...iconButtonProps}
       sx={sx}
       size="large"