From 46b57fe094816c394a6836a6357a9f8764e39058 Mon Sep 17 00:00:00 2001
From: ibolton336 <ibolton@redhat.com>
Date: Mon, 11 Dec 2023 10:46:46 -0500
Subject: [PATCH] :bug: Fix reports donut tooltip

Signed-off-by: ibolton336 <ibolton@redhat.com>
---
 .../application-landscape/application-landscape.tsx         | 4 ++++
 client/src/app/pages/reports/components/donut/donut.tsx     | 6 ++++--
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/client/src/app/pages/reports/components/application-landscape/application-landscape.tsx b/client/src/app/pages/reports/components/application-landscape/application-landscape.tsx
index 3c0107c833..f06ee70b0c 100644
--- a/client/src/app/pages/reports/components/application-landscape/application-landscape.tsx
+++ b/client/src/app/pages/reports/components/application-landscape/application-landscape.tsx
@@ -148,6 +148,7 @@ export const ApplicationLandscape: React.FC<IApplicationLandscapeProps> = ({
               riskLabel={
                 <Link to={getRisksUrl(["red"])}>{t("terms.highRisk")}</Link>
               }
+              riskTitle={t("terms.highRisk")}
               riskDescription={questionnaire?.riskMessages?.red ?? ""}
             />
           </FlexItem>
@@ -163,6 +164,7 @@ export const ApplicationLandscape: React.FC<IApplicationLandscapeProps> = ({
                   {t("terms.mediumRisk")}
                 </Link>
               }
+              riskTitle={t("terms.mediumRisk")}
               riskDescription={questionnaire?.riskMessages?.yellow ?? ""}
             />
           </FlexItem>
@@ -176,6 +178,7 @@ export const ApplicationLandscape: React.FC<IApplicationLandscapeProps> = ({
               riskLabel={
                 <Link to={getRisksUrl(["green"])}>{t("terms.lowRisk")}</Link>
               }
+              riskTitle={t("terms.lowRisk")}
               riskDescription={questionnaire?.riskMessages?.green ?? ""}
             />
           </FlexItem>
@@ -191,6 +194,7 @@ export const ApplicationLandscape: React.FC<IApplicationLandscapeProps> = ({
                   {`${t("terms.unassessed")}/${t("terms.unknown")}`}
                 </Link>
               }
+              riskTitle={t("terms.unassessed")}
             />
           </FlexItem>
         </Flex>
diff --git a/client/src/app/pages/reports/components/donut/donut.tsx b/client/src/app/pages/reports/components/donut/donut.tsx
index 8c1d413266..60b0ef040a 100644
--- a/client/src/app/pages/reports/components/donut/donut.tsx
+++ b/client/src/app/pages/reports/components/donut/donut.tsx
@@ -18,7 +18,8 @@ export interface IDonutProps {
   total: number;
   color: string;
   riskLabel: string | React.ReactElement;
-  riskDescription?: string | React.ReactElement;
+  riskDescription?: string;
+  riskTitle: string;
   isAssessment: boolean;
 }
 
@@ -29,6 +30,7 @@ export const Donut: React.FC<IDonutProps> = ({
   color,
   riskLabel,
   isAssessment,
+  riskTitle,
 }) => {
   const { t } = useTranslation();
 
@@ -50,7 +52,7 @@ export const Donut: React.FC<IDonutProps> = ({
             }
             constrainToVisibleArea={true}
             data={[
-              { x: riskLabel, y: value },
+              { x: riskTitle, y: value },
               { x: t("terms.other"), y: total - value },
             ]}
             labels={({ datum }) => `${datum.x}: ${datum.y}`}