Merge pull request #4124 from relative-ci/sort-by-actual-not-absolute-value-rltv-894

fix(ui): MetricsTable - sort by actual value

Author: vio

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';

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}
             >

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 = ({