Skip to content

Commit

Permalink
✨ Fix Report button for Suggested adoption plan (#1099)
Browse files Browse the repository at this point in the history
This will help with PF V5 migration.

Signed-off-by: Gilles Dubreuil <gdubreui@redhat.com>
  • Loading branch information
gildub authored Jul 6, 2023
1 parent 7130f44 commit a8e6de5
Showing 1 changed file with 13 additions and 17 deletions.
30 changes: 13 additions & 17 deletions client/src/app/pages/reports/reports.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@ import { useTranslation } from "react-i18next";

import {
Bullseye,
Button,
ButtonVariant,
Card,
CardActions,
CardBody,
Expand All @@ -22,6 +24,7 @@ import {
ToggleGroupItem,
} from "@patternfly/react-core";
import HelpIcon from "@patternfly/react-icons/dist/esm/icons/help-icon";
import spacing from "@patternfly/react-styles/css/utilities/Spacing/spacing";

import {
AppPlaceholder,
Expand Down Expand Up @@ -138,18 +141,10 @@ export const Reports: React.FC = () => {
<CardHeader
onExpand={() => setAdoptionPlanOpen((current) => !current)}
>
<CardTitle>
<Split style={{ marginTop: -5 }}>
<SplitItem>
<Bullseye style={{ marginTop: -3 }}>
<TextContent>
<Text component="h3">
{t("terms.suggestedAdoptionPlan")}
</Text>
</TextContent>
</Bullseye>
</SplitItem>
<SplitItem>
<CardTitle style={{ marginTop: -6 }}>
<TextContent>
<Text component="h3">
{t("terms.suggestedAdoptionPlan")}
<Popover
bodyContent={
<div>
Expand All @@ -158,17 +153,18 @@ export const Reports: React.FC = () => {
}
position="right"
>
<button
<Button
type="button"
aria-label="More info"
onClick={(e) => e.preventDefault()}
className="pf-c-button pf-m-plain"
isInline
variant={ButtonVariant.plain}
>
<HelpIcon />
</button>
</Button>
</Popover>
</SplitItem>
</Split>
</Text>
</TextContent>
</CardTitle>
</CardHeader>
<CardExpandableContent>
Expand Down

0 comments on commit a8e6de5

Please sign in to comment.