Merge pull request #6858 from LedgerHQ/fix/LIVE-8996

fix(lld): remove unnecessary information when the balance has not cha…
LedgerHQ · May 14, 2024 · 679bf73 · 679bf73
2 parents 77012ea + 50473ce
commit 679bf73
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 11 deletions.
diff --git a/.changeset/late-vans-lick.md b/.changeset/late-vans-lick.md
@@ -0,0 +1,5 @@
+---
+"ledger-live-desktop": patch
+---
+
+Remove 0 (currency) when balance hasn't change in the selected time range
diff --git a/apps/ledger-live-desktop/src/renderer/components/BalanceInfos/index.tsx b/apps/ledger-live-desktop/src/renderer/components/BalanceInfos/index.tsx
@@ -55,17 +55,21 @@ export function BalanceDiff({ valueChange, unit, isAvailable, ...boxProps }: Pro
             withIcon
           />
         )}
-        <FormattedVal
-          unit={unit}
-          val={valueChange.value}
-          prefix={valueChange.percentage ? " (" : undefined}
-          suffix={valueChange.percentage ? ")" : undefined}
-          withIcon={!valueChange.percentage}
-          alwaysShowSign={!!valueChange.percentage}
-          showCode
-          animateTicker
-          inline
-        />
+        {valueChange.value === 0 ? (
+          <Text color={"palette.text.shade100"}>{"-"}</Text>
+        ) : (
+          <FormattedVal
+            unit={unit}
+            val={valueChange.value}
+            prefix={valueChange.percentage ? " (" : undefined}
+            suffix={valueChange.percentage ? ")" : undefined}
+            withIcon={!valueChange.percentage}
+            alwaysShowSign={!!valueChange.percentage}
+            showCode
+            animateTicker
+            inline
+          />
+        )}
       </Box>
     </Box>
   );