Skip to content

Commit

Permalink
Fix one pixel of unclickable border in nav column
Browse files Browse the repository at this point in the history
  • Loading branch information
krschacht committed Nov 12, 2024
1 parent b003c45 commit 84e63a3
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
4 changes: 2 additions & 2 deletions app/views/assistants/_assistant.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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 %>
Expand Down
5 changes: 3 additions & 2 deletions app/views/layouts/_settings_item.erb
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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
Expand Down

0 comments on commit 84e63a3

Please sign in to comment.