diff --git a/webpack/components/AnsibleRolesSwitcher/components/AnsiblePermissionDenied.js b/webpack/components/AnsibleRolesSwitcher/components/AnsiblePermissionDenied.js index 8523fe24..34eae0cd 100644 --- a/webpack/components/AnsibleRolesSwitcher/components/AnsiblePermissionDenied.js +++ b/webpack/components/AnsibleRolesSwitcher/components/AnsiblePermissionDenied.js @@ -22,7 +22,6 @@ const AnsiblePermissionDenied = props => { return ( {}; const fixtures = { 'should render a role to add': { role: { name: 'test.role', id: 5 }, - icon: 'fa fa-plus-circle', + icon: 'plus-circle', onClick: noop, index: 0, }, 'should render a role to remove': { role: { name: 'test.role', id: 5 }, - icon: 'fa fa-minus-circle', + icon: 'minus-circle', onClick: noop, index: 0, }, 'should render inherited role to remove': { role: { name: 'test.role', id: 5, inherited: true }, - icon: 'fa fa-minus-circle', + icon: 'minus-circle', onClick: noop, index: 0, }, diff --git a/webpack/components/AnsibleRolesSwitcher/components/AnsibleRoleActionButton.js b/webpack/components/AnsibleRolesSwitcher/components/AnsibleRoleActionButton.js index 77f720f6..45338b85 100644 --- a/webpack/components/AnsibleRolesSwitcher/components/AnsibleRoleActionButton.js +++ b/webpack/components/AnsibleRolesSwitcher/components/AnsibleRoleActionButton.js @@ -1,13 +1,25 @@ import React from 'react'; import PropTypes from 'prop-types'; -import { Icon } from 'patternfly-react'; +import { Icon } from '@patternfly/react-core'; +import { PlusCircleIcon, MinusCircleIcon } from '@patternfly/react-icons'; -const AnsibleRoleActionButton = ({ icon }) => ( - -); +const iconMap = { + 'plus-circle': PlusCircleIcon, + 'minus-circle': MinusCircleIcon, +}; + +const AnsibleRoleActionButton = ({ icon }) => { + const IconComponent = iconMap[icon] || PlusCircleIcon; + + return ( + + ); +}; AnsibleRoleActionButton.propTypes = { icon: PropTypes.string.isRequired, diff --git a/webpack/components/AnsibleRolesSwitcher/components/AssignedRolesList.js b/webpack/components/AnsibleRolesSwitcher/components/AssignedRolesList.js index 2913f741..8b9f5f68 100644 --- a/webpack/components/AnsibleRolesSwitcher/components/AssignedRolesList.js +++ b/webpack/components/AnsibleRolesSwitcher/components/AssignedRolesList.js @@ -42,7 +42,7 @@ const AssignedRolesList = ({ key={role.id} role={role} index={idx} - icon="fa fa-minus-circle" + icon="minus-circle" onClick={onRemoveRole} resourceName={resourceName} /> @@ -54,7 +54,7 @@ const AssignedRolesList = ({ role={role} index={idx} moveValue={moveValue} - icon="fa fa-minus-circle" + icon="minus-circle" onClick={onRemoveRole} resourceName={resourceName} /> diff --git a/webpack/components/AnsibleRolesSwitcher/components/AvailableRolesList.js b/webpack/components/AnsibleRolesSwitcher/components/AvailableRolesList.js index 3bbc4691..928d34b9 100644 --- a/webpack/components/AnsibleRolesSwitcher/components/AvailableRolesList.js +++ b/webpack/components/AnsibleRolesSwitcher/components/AvailableRolesList.js @@ -29,7 +29,7 @@ const AvailableRolesList = ({ diff --git a/webpack/components/AnsibleRolesSwitcher/components/__snapshots__/AnsiblePermissionDenied.test.js.snap b/webpack/components/AnsibleRolesSwitcher/components/__snapshots__/AnsiblePermissionDenied.test.js.snap index 2f17d251..5fffddae 100644 --- a/webpack/components/AnsibleRolesSwitcher/components/__snapshots__/AnsiblePermissionDenied.test.js.snap +++ b/webpack/components/AnsibleRolesSwitcher/components/__snapshots__/AnsiblePermissionDenied.test.js.snap @@ -22,7 +22,7 @@ exports[`AnsiblePermissionDenied should render 1`] = ` } header="Permission Denied" icon="lock" - iconType="fa" secondaryActions={Array []} + variant="xl" /> `; diff --git a/webpack/components/AnsibleRolesSwitcher/components/__snapshots__/AnsibleRole.test.js.snap b/webpack/components/AnsibleRolesSwitcher/components/__snapshots__/AnsibleRole.test.js.snap index b3c52b81..bc85ea69 100644 --- a/webpack/components/AnsibleRolesSwitcher/components/__snapshots__/AnsibleRole.test.js.snap +++ b/webpack/components/AnsibleRolesSwitcher/components/__snapshots__/AnsibleRole.test.js.snap @@ -4,7 +4,7 @@ exports[`AnsibleRole should render a role to add 1`] = `