From ad557cd32ddf63bd5068b7a4977bfb4567547089 Mon Sep 17 00:00:00 2001 From: jenny-s51 Date: Mon, 10 Oct 2022 15:10:34 -0400 Subject: [PATCH] chore(card): remove isHoverable --- packages/react-core/src/components/Card/Card.tsx | 5 +---- .../react-core/src/components/Card/__tests__/Card.test.tsx | 5 ----- 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/packages/react-core/src/components/Card/Card.tsx b/packages/react-core/src/components/Card/Card.tsx index 129cf1aee32..36746cb75ce 100644 --- a/packages/react-core/src/components/Card/Card.tsx +++ b/packages/react-core/src/components/Card/Card.tsx @@ -12,8 +12,6 @@ export interface CardProps extends React.HTMLProps, OUIAProps { className?: string; /** Sets the base component to render. defaults to article */ component?: keyof JSX.IntrinsicElements; - /** @deprecated to make a card hoverable, use isSelectable or isSelectableRaised. */ - isHoverable?: boolean; /** Modifies the card to include compact styling. Should not be used with isLarge. */ isCompact?: boolean; /** Modifies the card to include selectable styling */ @@ -70,7 +68,6 @@ export const Card: React.FunctionComponent = ({ id = '', className = '', component = 'article', - isHoverable = false, isCompact = false, isSelectable = false, isSelectableRaised = false, @@ -107,7 +104,7 @@ export const Card: React.FunctionComponent = ({ if (isSelectableRaised) { return css(styles.modifiers.selectableRaised, isSelected && styles.modifiers.selectedRaised); } - if (isSelectable || isHoverable) { + if (isSelectable) { return css(styles.modifiers.selectable, isSelected && styles.modifiers.selected); } return ''; diff --git a/packages/react-core/src/components/Card/__tests__/Card.test.tsx b/packages/react-core/src/components/Card/__tests__/Card.test.tsx index a3abc450479..70f47666b5e 100644 --- a/packages/react-core/src/components/Card/__tests__/Card.test.tsx +++ b/packages/react-core/src/components/Card/__tests__/Card.test.tsx @@ -37,11 +37,6 @@ describe('Card', () => { expect(screen.getByText('im a div')).toBeInTheDocument(); }); - test('card with isHoverable applied ', () => { - const { asFragment } = render(); - expect(asFragment()).toMatchSnapshot(); - }); - test('card with isCompact applied ', () => { const { asFragment } = render(); expect(asFragment()).toMatchSnapshot();