From 8c87c14c5be19c6c5a2c6014409e65f881ba087c Mon Sep 17 00:00:00 2001
From: Keith Schacht <krschacht@gmail.com>
Date: Tue, 12 Nov 2024 17:48:32 -0600
Subject: [PATCH] Fix one pixel of unclickable border in nav column (#543)

---
 app/views/assistants/_assistant.html.erb | 4 ++--
 app/views/layouts/_settings_item.erb     | 5 +++--
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/app/views/assistants/_assistant.html.erb b/app/views/assistants/_assistant.html.erb
index d36f297f6..3e04f9fea 100644
--- a/app/views/assistants/_assistant.html.erb
+++ b/app/views/assistants/_assistant.html.erb
@@ -6,7 +6,7 @@
   <%# This extra div ^ is needed because of the absolute positioning. It doesn't lay out properly if added to the div below. %>
   <div class="
               flex justify-between items-center
-              mb-1 p-1 pl-2 pr-2 mr-5
+              mb-1 pl-2 pr-2 mr-5
               hover:bg-gray-100 dark:hover:bg-gray-700
               bg-gray-50 dark:bg-transparent
               dark:bg-gray-700
@@ -21,7 +21,7 @@
     data-radio-behavior-id-param="<%= assistant.id %>"
     data-transition-target="<%= !visible && 'transitionable' %>"
     >
-    <%= link_to new_assistant_message_path(assistant), class: "flex-1 flex items-center text-gray-950 dark:text-gray-100 font-medium truncate", data: { role: "name" } do %>
+    <%= link_to new_assistant_message_path(assistant), class: "flex-1 flex py-1 items-center text-gray-950 dark:text-gray-100 font-medium truncate", data: { role: "name" } do %>
       <%= render partial: "layouts/assistant_avatar", locals: { assistant: assistant, size: 7, classes: "mr-2" } %>
       <%= assistant.name %>
     <% end %>
diff --git a/app/views/layouts/_settings_item.erb b/app/views/layouts/_settings_item.erb
index 5ec2e5ecc..085de0b37 100644
--- a/app/views/layouts/_settings_item.erb
+++ b/app/views/layouts/_settings_item.erb
@@ -3,7 +3,7 @@
 <% selected = selected && params[:id] == item.try(:id)&.to_s if controller.to_s == "assistants" %>
 <div class="
             flex justify-between items-center
-            mb-1 p-1 pl-2
+            mb-1
             hover:bg-gray-100 dark:hover:bg-gray-700
             group
             text-sm rounded-s-lg
@@ -14,9 +14,10 @@
   <%= link_to link,
         class: %|
           flex-1 flex
+          p-1 pl-2
           items-center
           truncate
-          h-7
+          h-9
           #{selected && "cursor-default" }
         |,
         data: { role: "name" } do