From 2d492e1f68f91baa648e6c26dbaffa34c3746fca Mon Sep 17 00:00:00 2001
From: carkom <neilmccreery@yahoo.com>
Date: Thu, 26 Oct 2023 11:13:57 +0100
Subject: [PATCH] data display adjustments

---
 .../src/components/reports/ReportTable.tsx    | 43 +++++++++----------
 .../reports/graphs/StackedBarGraph.tsx        | 30 ++++++++-----
 2 files changed, 41 insertions(+), 32 deletions(-)

diff --git a/packages/desktop-client/src/components/reports/ReportTable.tsx b/packages/desktop-client/src/components/reports/ReportTable.tsx
index def27e33630..8e20b4d7698 100644
--- a/packages/desktop-client/src/components/reports/ReportTable.tsx
+++ b/packages/desktop-client/src/components/reports/ReportTable.tsx
@@ -47,6 +47,9 @@ const TableRow = memo(
           })}
         <Cell
           value={amountToCurrency(item[totalItem])}
+          style={{
+            fontWeight: 600,
+          }}
           width="flex"
           privacyFilter
         />
@@ -94,28 +97,24 @@ export function TableTotals({ data, scrollWidth, totalItem, mode }) {
       }}
     >
       <Cell value={'Totals'} width="flex" />
-      {mode === 'time' ? (
-        <>
-          {data.monthData.map(item => {
-            return (
-              <Cell
-                key={amountToCurrency(item[totalItem])}
-                value={amountToCurrency(item[totalItem])}
-                width="flex"
-                privacyFilter
-              />
-            );
-          })}
-          <Cell width="flex" />
-        </>
-      ) : (
-        <Cell
-          key={data[totalItem]}
-          value={amountToCurrency(data[totalItem])}
-          width="flex"
-          privacyFilter
-        />
-      )}
+      {mode === 'time' &&
+        data.monthData.map(item => {
+          return (
+            <Cell
+              key={amountToCurrency(item[totalItem])}
+              value={amountToCurrency(item[totalItem])}
+              width="flex"
+              privacyFilter
+            />
+          );
+        })}
+      <Cell
+        key={data[totalItem]}
+        value={amountToCurrency(data[totalItem])}
+        width="flex"
+        privacyFilter
+      />
+
       {scrollWidth > 0 && <Cell width={scrollWidth} />}
     </Row>
   );
diff --git a/packages/desktop-client/src/components/reports/graphs/StackedBarGraph.tsx b/packages/desktop-client/src/components/reports/graphs/StackedBarGraph.tsx
index 09e33fb36b9..d53acadfc06 100644
--- a/packages/desktop-client/src/components/reports/graphs/StackedBarGraph.tsx
+++ b/packages/desktop-client/src/components/reports/graphs/StackedBarGraph.tsx
@@ -60,12 +60,6 @@ function StackedBarGraph({
     };
   };
 
-  type CustomTooltipProps = {
-    active?: boolean;
-    payload?: PayloadItem[];
-    label?: string;
-  };
-
   type CustomLegendProps = {
     active?: boolean;
     payload?: PayloadItem[];
@@ -85,8 +79,15 @@ function StackedBarGraph({
     return <div />;
   };
 
+  type CustomTooltipProps = {
+    active?: boolean;
+    payload?: PayloadItem[];
+    label?: string;
+  };
+
   const CustomTooltip = ({ active, payload, label }: CustomTooltipProps) => {
     if (active && payload && payload.length) {
+      let sumTotals = 0;
       return (
         <div
           className={`${css(
@@ -111,8 +112,9 @@ function StackedBarGraph({
                 {payload
                   .slice(0)
                   .reverse()
-                  .map(
-                    pay =>
+                  .map(pay => {
+                    sumTotals += pay.value;
+                    return (
                       pay.value !== 0 && (
                         <AlignedText
                           key={pay.name}
@@ -120,8 +122,16 @@ function StackedBarGraph({
                           right={amountToCurrency(pay.value)}
                           style={{ color: pay.color }}
                         />
-                      ),
-                  )}
+                      )
+                    );
+                  })}
+                <AlignedText
+                  left={'Total'}
+                  right={amountToCurrency(sumTotals)}
+                  style={{
+                    fontWeight: 600,
+                  }}
+                />
               </PrivacyFilter>
             </div>
           </div>