Skip to content

Commit 910dd72

Browse files
committed
fix(MenuItemAction): update to use new favorite button props
1 parent 8e9e603 commit 910dd72

1 file changed

Lines changed: 4 additions & 11 deletions

File tree

packages/react-core/src/components/Menu/MenuItemAction.tsx

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import { forwardRef } from 'react';
22
import styles from '@patternfly/react-styles/css/components/Menu/menu';
33
import { css } from '@patternfly/react-styles';
4-
import StarIcon from '@patternfly/react-icons/dist/esm/icons/star-icon';
54
import { MenuContext, MenuItemContext } from './MenuContext';
65
import { Button } from '../Button';
76
export interface MenuItemActionProps extends React.HTMLProps<HTMLDivElement> {
@@ -45,24 +44,18 @@ const MenuItemActionBase: React.FunctionComponent<MenuItemActionProps> = ({
4544
onActionClick && onActionClick(event, itemId, actionId);
4645
};
4746
return (
48-
<div
49-
className={css(
50-
styles.menuItemAction,
51-
isFavorited !== null && 'pf-m-favorite',
52-
isFavorited && styles.modifiers.favorited,
53-
className
54-
)}
55-
{...props}
56-
>
47+
<div className={css(styles.menuItemAction, className)} {...props}>
5748
<Button
5849
aria-label={ariaLabel}
5950
onClick={onClickButton}
6051
ref={innerRef}
6152
role="menuitem"
6253
variant="plain"
54+
isFavorite={isFavorited !== null}
55+
isFavorited={isFavorited ?? false}
6356
tabIndex={-1}
6457
isDisabled={isDisabled || isDisabledContext}
65-
icon={icon === 'favorites' || isFavorited !== null ? <StarIcon /> : icon}
58+
icon={isFavorited === null ? icon : undefined}
6659
/>
6760
</div>
6861
);

0 commit comments

Comments
 (0)