fix: review

Author: Raubzeug

src/utils/dataFormatters/__test__/formatNumbers.test.ts

@@ -26,19 +26,14 @@ describe('formatNumericValues', () => {
         expect(result).toEqual([`1${UNBREAKABLE_GAP}k`, `2${UNBREAKABLE_GAP}k`]);
     });
 
-    it('should handle small value compared to total and increase precision', () => {
-        const result = formatNumericValues(1, 1024);
-        expect(result).toEqual(['0.001', `1${UNBREAKABLE_GAP}k`]);
-    });
-
     it('should return ["0", formattedTotal] when value is 0', () => {
         const result = formatNumericValues(0, 2048);
         expect(result).toEqual(['0', `2${UNBREAKABLE_GAP}k`]);
     });
 
     it('should use provided size and delimiter', () => {
         const result = formatNumericValues(5120, 10240, 'billion', '/');
-        expect(result).toEqual(['0.00001', '0/b']);
+        expect(result).toEqual(['0', '0/b']);
     });
 
     it('should handle non-numeric total gracefully', () => {

src/utils/dataFormatters/__test__/formatStorageValues.test.ts

@@ -22,19 +22,14 @@ describe('formatStorageValues', () => {
         expect(result).toEqual(['1', `2${UNBREAKABLE_GAP}KB`]);
     });
 
-    it('should handle small value compared to total and increase precision', () => {
-        const result = formatStorageValues(1, 1024);
-        expect(result).toEqual(['0.001', `1${UNBREAKABLE_GAP}KB`]);
-    });
-
     it('should return ["0", formattedTotal] when value is 0', () => {
         const result = formatStorageValues(0, 2048);
         expect(result).toEqual(['0', `2${UNBREAKABLE_GAP}KB`]);
     });
 
     it('should use provided size and delimiter', () => {
         const result = formatStorageValues(5120, 10240, 'mb', '/');
-        expect(result).toEqual(['0.01', '0/MB']);
+        expect(result).toEqual(['0', '0/MB']);
     });
 
     it('should handle non-numeric total gracefully', () => {

src/utils/dataFormatters/common.ts

@@ -33,25 +33,13 @@ export function formatValues<T>(
         valuePrecision = 1;
     }
 
-    let formattedValue = formatter({
+    const formattedValue = formatter({
         value,
         withSizeLabel: valueWithSizeLabel,
         size: size || calculatedSize,
         precision: valuePrecision,
         delimiter,
     });
-    if (value && value > 0) {
-        while (parseFloat(formattedValue) === 0) {
-            valuePrecision += 1;
-            formattedValue = formatter({
-                value,
-                withSizeLabel: valueWithSizeLabel,
-                size: size || calculatedSize,
-                precision: valuePrecision,
-                delimiter,
-            });
-        }
-    }
     const formattedTotal = formatter({
         value: total,
         size: size || calculatedSize,

src/utils/dataFormatters/dataFormatters.ts

@@ -113,11 +113,7 @@ export const formatPercent = (number?: unknown) => {
         return '';
     }
     const configuredNumber = configuredNumeral(number);
-    const numberValue = configuredNumber.value();
-    let format = '0.[0]%';
-    if (numberValue && numberValue < 0.001) {
-        format = '0.[00]%';
-    }
+    const format = '0%';
     return configuredNumber.format(format);
 };