From aa02a45962f5873b5d470fb606df1d6e5d24e3fb Mon Sep 17 00:00:00 2001
From: Thomas Kaul <4159106+dtslvr@users.noreply.github.com>
Date: Tue, 4 Aug 2026 18:50:02 +0200
Subject: [PATCH] Improve usability by collapsing breakdowns

---
 .../portfolio-summary/portfolio-summary.component.html    | 8 ++++++--
 .../portfolio-summary/portfolio-summary.component.scss    | 4 ----
 .../portfolio-summary/portfolio-summary.component.ts      | 4 ++--
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/apps/client/src/app/components/portfolio-summary/portfolio-summary.component.html b/apps/client/src/app/components/portfolio-summary/portfolio-summary.component.html
index 5e5205582..9394d7624 100644
--- a/apps/client/src/app/components/portfolio-summary/portfolio-summary.component.html
+++ b/apps/client/src/app/components/portfolio-summary/portfolio-summary.component.html
@@ -170,7 +170,9 @@
       (click)="onToggleHoldings()"
     >
       @if (hasHoldingsBreakdown) {
-        <span class="caret-container">
+        <span
+          class="align-items-center caret-container d-inline-flex justify-content-center ml-n3"
+        >
           <ion-icon
             class="caret text-muted"
             name="caret-forward-outline"
@@ -242,7 +244,9 @@
       (click)="onToggleCash()"
     >
       @if (hasCashBreakdown) {
-        <span class="caret-container">
+        <span
+          class="align-items-center caret-container d-inline-flex justify-content-center ml-n3"
+        >
           <ion-icon
             class="caret text-muted"
             name="caret-forward-outline"
diff --git a/apps/client/src/app/components/portfolio-summary/portfolio-summary.component.scss b/apps/client/src/app/components/portfolio-summary/portfolio-summary.component.scss
index 0b8156f95..534976f11 100644
--- a/apps/client/src/app/components/portfolio-summary/portfolio-summary.component.scss
+++ b/apps/client/src/app/components/portfolio-summary/portfolio-summary.component.scss
@@ -2,10 +2,6 @@
   display: block;
 
   .caret-container {
-    align-items: center;
-    display: inline-flex;
-    justify-content: center;
-    margin-left: -1rem;
     width: 1rem;
 
     .caret {
diff --git a/apps/client/src/app/components/portfolio-summary/portfolio-summary.component.ts b/apps/client/src/app/components/portfolio-summary/portfolio-summary.component.ts
index 0213948ec..eac5ef9a6 100644
--- a/apps/client/src/app/components/portfolio-summary/portfolio-summary.component.ts
+++ b/apps/client/src/app/components/portfolio-summary/portfolio-summary.component.ts
@@ -85,11 +85,11 @@ export class GfPortfolioSummaryComponent implements OnChanges {
   }
 
   protected get hasCashBreakdown() {
-    return this.isLoading || this.summary?.emergencyFund?.cash > 0;
+    return !this.isLoading && this.summary?.emergencyFund?.cash > 0;
   }
 
   protected get hasHoldingsBreakdown() {
-    return this.isLoading || this.summary?.emergencyFund?.assets > 0;
+    return !this.isLoading && this.summary?.emergencyFund?.assets > 0;
   }
 
   protected get holdingsInBaseCurrency() {