From 92cd23307698ffcbf89b387da1974dd4c1e61879 Mon Sep 17 00:00:00 2001
From: Vio <vio@beanon.com>
Date: Sat, 30 Dec 2023 15:20:14 +0100
Subject: [PATCH] fix(ui): MetricsTable - sort by actual value

---
 .../ui/src/components/metrics-table/metrics-table.i18n.ts   | 6 +++---
 packages/ui/src/components/metrics-table/metrics-table.tsx  | 6 +++---
 packages/ui/src/hooks/rows-sort.ts                          | 2 +-
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/packages/ui/src/components/metrics-table/metrics-table.i18n.ts b/packages/ui/src/components/metrics-table/metrics-table.i18n.ts
index 65839a1055..ed5cbe08fb 100644
--- a/packages/ui/src/components/metrics-table/metrics-table.i18n.ts
+++ b/packages/ui/src/components/metrics-table/metrics-table.i18n.ts
@@ -1,9 +1,9 @@
 export const BASELINE_TITLE = 'Baseline';
 export const CURRENT_TITLE = 'Current';
 
-export const SORT_ABSOLUTE_VALUE = 'absolute value';
-export const SORT_ABSOLUTE_CHANGE = 'absolute change';
-export const SORT_PERCENTUAL_CHANGE = 'absolute percentual change';
+export const SORT_VALUE = 'value';
+export const SORT_CHANGE = 'change';
+export const SORT_PERCENTUAL_CHANGE = 'percentual change';
 
 export const EMPTY_MESSAGE = 'No items found.';
 export const SHOW_ALL = 'Show all';
diff --git a/packages/ui/src/components/metrics-table/metrics-table.tsx b/packages/ui/src/components/metrics-table/metrics-table.tsx
index 73656d5694..131e1a05e2 100644
--- a/packages/ui/src/components/metrics-table/metrics-table.tsx
+++ b/packages/ui/src/components/metrics-table/metrics-table.tsx
@@ -92,7 +92,7 @@ const SumColumn = ({ rows, isBaseline, runIndex, updateSort, sort }: ColumnSumPr
         <SortButton
           fieldPath={fieldPath}
           fieldName="value"
-          label="absolute value"
+          label={I18N.SORT_VALUE}
           updateSort={updateSort}
           sort={sort}
         >
@@ -105,7 +105,7 @@ const SumColumn = ({ rows, isBaseline, runIndex, updateSort, sort }: ColumnSumPr
             <SortButton
               fieldPath={fieldPath}
               fieldName="delta"
-              label="absolute change"
+              label={I18N.SORT_CHANGE}
               updateSort={updateSort}
               sort={sort}
             >
@@ -118,7 +118,7 @@ const SumColumn = ({ rows, isBaseline, runIndex, updateSort, sort }: ColumnSumPr
             <SortButton
               fieldPath={fieldPath}
               fieldName="deltaPercentage"
-              label="absolute percentual change"
+              label={I18N.SORT_PERCENTUAL_CHANGE}
               updateSort={updateSort}
               sort={sort}
             >
diff --git a/packages/ui/src/hooks/rows-sort.ts b/packages/ui/src/hooks/rows-sort.ts
index 907b9b4426..001b2ee82d 100644
--- a/packages/ui/src/hooks/rows-sort.ts
+++ b/packages/ui/src/hooks/rows-sort.ts
@@ -18,7 +18,7 @@ export const getSortFn =
       return getCustomSort(item);
     }
 
-    return Math.abs(get(item, fieldPath) || 0);
+    return get(item, fieldPath) || 0;
   };
 
 export const useRowsSort = ({