From 5f8a0dc78eae3f2c6390d5216eefa901e5dc263d Mon Sep 17 00:00:00 2001
From: Alessio Cappa <104093777+alessiocappa@users.noreply.github.com>
Date: Tue, 13 Jan 2026 13:23:10 +0100
Subject: [PATCH] FIX: Small layout adjustments (#507)

* feat: adjust padding and icon sizing in outflows chart

* fix: Remove extra div in reports page to fix layout issues on mobile

* fix: Padding adjustments to keep pages consistent

* fix: Align report page with main
---
 app/views/pages/dashboard.html.erb                 | 2 +-
 app/views/pages/dashboard/_outflows_donut.html.erb | 4 ++--
 app/views/reports/_trends_insights.html.erb        | 4 ++--
 app/views/reports/index.html.erb                   | 2 +-
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/app/views/pages/dashboard.html.erb b/app/views/pages/dashboard.html.erb
index daf6454b4..3b51717e8 100644
--- a/app/views/pages/dashboard.html.erb
+++ b/app/views/pages/dashboard.html.erb
@@ -27,7 +27,7 @@
   </div>
 <% end %>
 
-<div class="w-full space-y-6 pb-24" data-controller="dashboard-sortable" data-action="dragover->dashboard-sortable#dragOver drop->dashboard-sortable#drop" role="list" aria-label="Dashboard sections">
+<div class="w-full space-y-6 pb-6 lg:pb-12" data-controller="dashboard-sortable" data-action="dragover->dashboard-sortable#dragOver drop->dashboard-sortable#drop" role="list" aria-label="Dashboard sections">
   <% if Current.family.accounts.any? %>
     <% @dashboard_sections.each do |section| %>
       <% next unless section[:visible] %>
diff --git a/app/views/pages/dashboard/_outflows_donut.html.erb b/app/views/pages/dashboard/_outflows_donut.html.erb
index ac09ae4d4..882a344f6 100644
--- a/app/views/pages/dashboard/_outflows_donut.html.erb
+++ b/app/views/pages/dashboard/_outflows_donut.html.erb
@@ -13,7 +13,7 @@
   <div class="px-4">
     <div class="flex flex-col lg:flex-row gap-8 items-center">
       <!-- Donut Chart -->
-      <div class="w-full lg:w-1/3 max-w-[300px] p-4">
+      <div class="w-full lg:w-1/3 max-w-[300px] px-4">
         <div class="h-[300px] relative"
           data-controller="donut-chart"
           data-donut-chart-segments-value="<%= outflows_data[:categories].to_json %>"
@@ -74,7 +74,7 @@
               category_content = capture do
             %>
               <div class="flex items-center gap-3 flex-1 min-w-0">
-                <div class="h-6 w-6 flex-shrink-0 group-hover:scale-105 transition-all duration-300 rounded-full flex justify-center items-center"
+                <div class="h-7 w-7 flex-shrink-0 group-hover:scale-105 transition-all duration-300 rounded-full flex justify-center items-center" 
                   style="
                     background-color: color-mix(in oklab, <%= category[:color] %> 10%, transparent);
                     border-color: color-mix(in oklab, <%= category[:color] %> 10%, transparent);
diff --git a/app/views/reports/_trends_insights.html.erb b/app/views/reports/_trends_insights.html.erb
index ad95b15c4..9ad002c94 100644
--- a/app/views/reports/_trends_insights.html.erb
+++ b/app/views/reports/_trends_insights.html.erb
@@ -1,4 +1,4 @@
-<div>
+<div class="space-y-8">
     <%# Month-over-Month Trends %>
     <div>
       <h3 class="text-sm font-medium text-secondary mb-4">
@@ -78,4 +78,4 @@
         </div>
       <% end %>
     </div>
-  </div>
+  </div>
\ No newline at end of file
diff --git a/app/views/reports/index.html.erb b/app/views/reports/index.html.erb
index 8940b7551..73b7dfcfa 100644
--- a/app/views/reports/index.html.erb
+++ b/app/views/reports/index.html.erb
@@ -96,7 +96,7 @@
   </div>
 <% end %>
 
-<div class="w-full space-y-6 pb-24">
+<div class="w-full space-y-6 pb-4 lg:pb-6">
   <% if Current.family.transactions.any? %>
     <%# Summary Dashboard - Always visible, not collapsible %>
     <section>