Replace text-danger with text-destructive in reports views (#444)

* Replace text-danger with text-destructive in reports views

Updated multiple report partials to use the 'text-destructive' class instead of 'text-danger' as this does not exist. This affects budget performance, summary dashboard, transactions breakdown, and trends insights views.

* Replace bg-danger with bg-destructive in report views

app/views/reports/_budget_performance.html.erb

@@ -22,7 +22,7 @@
             <div class="flex items-center gap-3">
               <% case budget_item[:status] %>
               <% when :over %>
-                <span class="inline-flex items-center gap-1 px-2 py-1 bg-danger/10 text-danger text-xs font-medium rounded-full">
+                <span class="inline-flex items-center gap-1 px-2 py-1 bg-destructive/10 text-destructive text-xs font-medium rounded-full">
                   <%= icon("alert-circle", class: "w-3 h-3") %>
                   <%= t("reports.budget_performance.status.over") %>
                 </span>
@@ -49,7 +49,7 @@
             <div class="h-3 bg-container rounded-full overflow-hidden">
               <% bar_width = [budget_item[:percent_used], 100].min %>
               <% bar_color = case budget_item[:status]
-                when :over then "bg-danger"
+                when :over then "bg-destructive"
                 when :warning then "bg-warning"
                 else "bg-success"
               end %>
@@ -83,7 +83,7 @@
                 </span>
               <% else %>
                 <span class="text-tertiary"><%= t("reports.budget_performance.over_by") %>:</span>
-                <span class="font-medium text-danger">
+                <span class="font-medium text-destructive">
                   <%= Money.new(budget_item[:remaining].abs, Current.family.currency).format %>
                 </span>
               <% end %>

app/views/reports/_summary_dashboard.html.erb

@@ -23,8 +23,8 @@
               +<%= metrics[:income_change] %>%
             </span>
           <% else %>
-            <%= icon("arrow-down", class: "w-4 h-4 text-danger") %>
-            <span class="text-sm font-medium text-danger">
+            <%= icon("arrow-down", class: "w-4 h-4 text-destructive") %>
+            <span class="text-sm font-medium text-destructive">
               <%= metrics[:income_change] %>%
             </span>
           <% end %>
@@ -40,7 +40,7 @@
   <div class="bg-container rounded-xl shadow-border-xs p-6">
     <div class="flex items-start justify-between mb-3">
       <div class="flex items-center gap-2">
-        <%= icon("trending-down", class: "w-5 h-5 text-danger") %>
+        <%= icon("trending-down", class: "w-5 h-5 text-destructive") %>
         <h3 class="text-sm font-medium text-secondary">
           <%= t("reports.summary.total_expenses") %>
         </h3>
@@ -55,8 +55,8 @@
       <% if metrics[:expense_change] %>
         <div class="flex items-center gap-1.5">
           <% if metrics[:expense_change] >= 0 %>
-            <%= icon("arrow-up", class: "w-4 h-4 text-danger") %>
-            <span class="text-sm font-medium text-danger">
+            <%= icon("arrow-up", class: "w-4 h-4 text-destructive") %>
+            <span class="text-sm font-medium text-destructive">
               +<%= metrics[:expense_change] %>%
             </span>
           <% else %>
@@ -85,7 +85,7 @@
     </div>
 
     <div class="space-y-2">
-      <p class="text-2xl font-semibold <%= metrics[:net_savings] >= 0 ? "text-success" : "text-danger" %>">
+      <p class="text-2xl font-semibold <%= metrics[:net_savings] >= 0 ? "text-success" : "text-destructive" %>">
         <%= metrics[:net_savings].format %>
       </p>
 
@@ -114,7 +114,7 @@
 
         <div class="space-y-1.5">
           <div class="h-2 bg-surface-inset rounded-full overflow-hidden">
-            <div class="h-full <%= metrics[:budget_percent] >= 100 ? "bg-danger" : metrics[:budget_percent] >= 80 ? "bg-warning" : "bg-success" %> rounded-full transition-all"
+            <div class="h-full <%= metrics[:budget_percent] >= 100 ? "bg-destructive" : metrics[:budget_percent] >= 80 ? "bg-warning" : "bg-success" %> rounded-full transition-all"
                  style="width: <%= [metrics[:budget_percent], 100].min %>%"></div>
           </div>
 

app/views/reports/_transactions_breakdown.html.erb

@@ -109,7 +109,7 @@
       <%# Expenses Section %>
       <% if expense_groups.any? %>
         <div>
-          <h3 class="text-base font-semibold text-danger mb-4 flex items-center gap-2">
+          <h3 class="text-base font-semibold text-destructive mb-4 flex items-center gap-2">
             <%= icon("trending-down", class: "w-5 h-5") %>
             <%= t("reports.transactions_breakdown.table.expense") %>
             <span class="text-sm font-normal text-tertiary">(<%= Money.new(expense_total, Current.family.currency).format %>)</span>
@@ -143,7 +143,7 @@
                       </div>
                     </td>
                     <td class="py-3 px-4 text-right">
-                      <span class="font-semibold text-danger">
+                      <span class="font-semibold text-destructive">
                         <%= Money.new(group[:total], Current.family.currency).format %>
                       </span>
                     </td>

app/views/reports/_trends_insights.html.erb

@@ -29,13 +29,13 @@
                   <td class="text-right py-3 px-4 text-success">
                     <%= Money.new(trend[:income], Current.family.currency).format %>
                   </td>
-                  <td class="text-right py-3 px-4 text-danger">
+                  <td class="text-right py-3 px-4 text-destructive">
                     <%= Money.new(trend[:expenses], Current.family.currency).format %>
                   </td>
-                  <td class="text-right py-3 px-2 <%= trend[:net] >= 0 ? "text-success" : "text-danger" %>">
+                  <td class="text-right py-3 px-2 <%= trend[:net] >= 0 ? "text-success" : "text-destructive" %>">
                     <%= Money.new(trend[:net], Current.family.currency).format %>
                   </td>
-                  <td class="text-right py-3 pl-4 <%= trend[:net] >= 0 ? "text-success" : "text-danger" %>">
+                  <td class="text-right py-3 pl-4 <%= trend[:net] >= 0 ? "text-success" : "text-destructive" %>">
                     <% savings_rate = trend[:income] > 0 ? ((trend[:net].to_f / trend[:income].to_f) * 100).round(1) : 0 %>
                     <%= savings_rate %>%
                   </td>
@@ -60,14 +60,14 @@
 
           <div class="p-4 bg-surface-inset rounded-lg">
             <p class="text-xs text-tertiary mb-1"><%= t("reports.trends.avg_monthly_expenses") %></p>
-            <p class="text-lg font-semibold text-danger">
+            <p class="text-lg font-semibold text-destructive">
               <%= Money.new(avg_expenses, Current.family.currency).format %>
             </p>
           </div>
 
           <div class="p-4 bg-surface-inset rounded-lg">
             <p class="text-xs text-tertiary mb-1"><%= t("reports.trends.avg_monthly_savings") %></p>
-            <p class="text-lg font-semibold <%= avg_net >= 0 ? "text-success" : "text-danger" %>">
+            <p class="text-lg font-semibold <%= avg_net >= 0 ? "text-success" : "text-destructive" %>">
               <%= Money.new(avg_net, Current.family.currency).format %>
             </p>
           </div>