fix(plugins/plugin-client-common): reverse the dependency of Card and DropDown

Fixes #4641

Author: myan9

plugins/plugin-client-common/src/components/spi/Card/impl/PatternFly.tsx

@@ -15,8 +15,6 @@
  */
 
 import * as React from 'react'
-import Props, { Action } from '../model'
-
 import {
   Card,
   CardActions,
@@ -28,6 +26,9 @@ import {
   KebabToggle
 } from '@patternfly/react-core'
 
+import Props from '../model'
+import { DropDownAction } from '../../DropDown'
+
 import '../../../../../web/scss/components/Card/Patternfly.scss'
 
 interface State {
@@ -43,7 +44,7 @@ export default class PatternflyCard extends React.PureComponent<Props, State> {
     }
   }
 
-  private renderDropDownItems(actions: Action[]) {
+  private renderDropDownItems(actions: DropDownAction[]) {
     return actions.map((item, idx) => (
       <DropdownItem key={idx} component="button" onClick={item.handler} data-mode={item.label}>
         {item.label}

plugins/plugin-client-common/src/components/spi/Card/model.ts

@@ -16,10 +16,7 @@
 
 import { ReactNode } from 'react'
 
-export interface Action {
-  label: string
-  handler: () => void
-}
+import { DropDownAction } from '../DropDown'
 
 interface Props {
   /** Place the given header node at the top of the Card */
@@ -29,7 +26,7 @@ interface Props {
   children: ReactNode
 
   /** [Optional] Actions to be rendered in an overflow menu associated with the Card */
-  actions?: Action[]
+  actions?: DropDownAction[]
 
   /** [Optional] CSS class to associate with the outermost Card element */
   className?: string

plugins/plugin-client-common/src/components/spi/DropDown/model.ts

@@ -14,8 +14,10 @@
  * limitations under the License.
  */
 
-import { Action } from '../Card/model'
-export { Action }
+export interface Action {
+  label: string
+  handler: () => void
+}
 
 interface Props {
   /** Actions to be rendered in an overflow menu associated with the Card */