From 5dfd172b29a0e796e4dea234ffa9ba78620e9765 Mon Sep 17 00:00:00 2001
From: Alessio Cappa <alessiocappa26@gmail.com>
Date: Sat, 13 Dec 2025 13:43:21 +0100
Subject: [PATCH] fix: Adjust mobile paddings

---
 app/components/UI/account/activity_feed.html.erb | 2 +-
 app/views/accounts/_logo.html.erb                | 2 ++
 app/views/accounts/show/_activity.html.erb       | 2 +-
 app/views/shared/_ruler.html.erb                 | 2 +-
 app/views/transactions/_transaction.html.erb     | 7 ++++++-
 app/views/transactions/index.html.erb            | 2 +-
 app/views/valuations/_valuation.html.erb         | 6 +++---
 7 files changed, 15 insertions(+), 8 deletions(-)

diff --git a/app/components/UI/account/activity_feed.html.erb b/app/components/UI/account/activity_feed.html.erb
index 82d164f9c..3aa1d9f51 100644
--- a/app/components/UI/account/activity_feed.html.erb
+++ b/app/components/UI/account/activity_feed.html.erb
@@ -1,5 +1,5 @@
 <%= turbo_frame_tag dom_id(account, "entries") do %>
-  <div class="bg-container p-5 shadow-border-xs rounded-xl" data-controller="checkbox-toggle">
+  <div class="bg-container px-3 py-4 lg:p-4 shadow-border-xs rounded-xl" data-controller="checkbox-toggle">
     <div class="flex items-center justify-between mb-4" data-testid="activity-menu">
       <%= tag.h2 t("accounts.show.activity.title"), class: "font-medium text-lg" %>
 
diff --git a/app/views/accounts/_logo.html.erb b/app/views/accounts/_logo.html.erb
index 364e7ccb5..494d09061 100644
--- a/app/views/accounts/_logo.html.erb
+++ b/app/views/accounts/_logo.html.erb
@@ -9,6 +9,8 @@
 
 <% if account.linked? && account.institution_domain.present? && Setting.brand_fetch_client_id.present? %>
   <%= image_tag "https://cdn.brandfetch.io/#{account.institution_domain}/icon/fallback/lettermark/w/40/h/40?c=#{Setting.brand_fetch_client_id}", class: "shrink-0 rounded-full #{size_classes[size]}" %>
+<% elsif account.enable_banking_account_id? && (enable_banking_account = EnableBankingAccount.find_by(id: account.enable_banking_account_id)) && (enable_banking_item = EnableBankingItem.find_by(id: enable_banking_account.enable_banking_item_id)) %>
+  <%= image_tag "#{enable_banking_item.logo_url}", class: "shrink-0 rounded-full #{size_classes[size]} h-auto" %>
 <% elsif account.logo.attached? %>
   <%= image_tag account.logo, class: "shrink-0 rounded-full #{size_classes[size]}" %>
 <% else %>
diff --git a/app/views/accounts/show/_activity.html.erb b/app/views/accounts/show/_activity.html.erb
index cab1760e0..3ea269609 100644
--- a/app/views/accounts/show/_activity.html.erb
+++ b/app/views/accounts/show/_activity.html.erb
@@ -1,7 +1,7 @@
 <%# locals: (account:) %>
 
 <%= turbo_frame_tag dom_id(account, "entries") do %>
-  <div class="bg-container p-5 shadow-border-xs rounded-xl">
+  <div class="bg-container px-3 py-4 lg:p-4 shadow-border-xs rounded-xl">
     <div class="flex items-center justify-between mb-4" data-testid="activity-menu">
       <%= tag.h2 t(".title"), class: "font-medium text-lg" %>
       <% unless @account.linked? %>
diff --git a/app/views/shared/_ruler.html.erb b/app/views/shared/_ruler.html.erb
index faf96d8ee..2c1ed15aa 100644
--- a/app/views/shared/_ruler.html.erb
+++ b/app/views/shared/_ruler.html.erb
@@ -1,2 +1,2 @@
 <%# locals: (classes: nil) %>
-<hr class="border-divider <%= classes || "mx-4" %>">
+<hr class="border-divider <%= classes || "mx-3 lg:mx-4" %>">
diff --git a/app/views/transactions/_transaction.html.erb b/app/views/transactions/_transaction.html.erb
index 2a0d70cdf..cb54c49a2 100644
--- a/app/views/transactions/_transaction.html.erb
+++ b/app/views/transactions/_transaction.html.erb
@@ -89,7 +89,12 @@
                           data: { turbo_frame: "_top" },
                           class: "hover:underline" %>
                     </span>
-                    <%= render "categories/category_name_mobile", transaction: transaction %>
+                    <div class="flex items-center gap-1">
+                      <%= render "categories/category_name_mobile", transaction: transaction %>
+                      <% if transaction.merchant&.present? %>
+                        <span class="lg:hidden truncate">• <%= transaction.merchant.name %></span>
+                      <% end %>
+                    </div>
                   <% end %>
                 </div>
               </div>
diff --git a/app/views/transactions/index.html.erb b/app/views/transactions/index.html.erb
index ea9f8b5d9..5f5b889bf 100644
--- a/app/views/transactions/index.html.erb
+++ b/app/views/transactions/index.html.erb
@@ -49,7 +49,7 @@
        data-controller="bulk-select checkbox-toggle"
        data-bulk-select-singular-label-value="<%= t(".transaction") %>"
        data-bulk-select-plural-label-value="<%= t(".transactions") %>"
-       class="flex flex-col bg-container rounded-xl shadow-border-xs p-4">
+       class="flex flex-col bg-container rounded-xl shadow-border-xs px-3 py-4 lg:p-4">
     <%= render "transactions/searches/search" %>
 
     <div id="entry-selection-bar" data-bulk-select-target="selectionBar" class="flex justify-center hidden">
diff --git a/app/views/valuations/_valuation.html.erb b/app/views/valuations/_valuation.html.erb
index 4fbff9cc6..201628ff1 100644
--- a/app/views/valuations/_valuation.html.erb
+++ b/app/views/valuations/_valuation.html.erb
@@ -7,14 +7,14 @@
 
 <%= turbo_frame_tag dom_id(entry) do %>
   <%= turbo_frame_tag dom_id(valuation) do %>
-    <div class="p-4 grid grid-cols-12 items-center text-primary text-sm font-medium">
-      <div class="col-span-8 flex items-center gap-4">
+    <div class="p-3 lg:p-4 grid grid-cols-12 items-center text-primary text-sm font-medium">
+      <div class="col-span-8 flex items-center gap-3 lg:gap-4">
         <%= check_box_tag dom_id(entry, "selection"),
                         class: "checkbox checkbox--light hidden lg:block",
                         data: { id: entry.id, "bulk-select-target": "row", action: "bulk-select#toggleRowSelection", checkbox_toggle_target: "selectionEntry" } %>
 
         <div class="flex items-center gap-3">
-          <%= render DS::FilledIcon.new(icon: icon, size: "sm", hex_color: color, rounded: true) %>
+          <%= render DS::FilledIcon.new(icon: icon, size: "md", hex_color: color, rounded: true) %>
 
           <div class="truncate text-primary">
             <%= link_to entry.name,