From dafcd0ed6fd4477de9a01125e87ba000356e619e Mon Sep 17 00:00:00 2001
From: Claudio Ortolina <cloud8421@gmail.com>
Date: Tue, 15 Oct 2024 14:58:14 +0100
Subject: [PATCH] Fix side padding inconsistencies in collection view

---
 .../components/core_components.ex             |  2 +-
 .../components/pagination.ex                  |  5 +-
 .../live/record_live/index.html.heex          | 54 ++++++++++---------
 3 files changed, 30 insertions(+), 31 deletions(-)

diff --git a/lib/music_library_web/components/core_components.ex b/lib/music_library_web/components/core_components.ex
index 6f598e68..0ca996f2 100644
--- a/lib/music_library_web/components/core_components.ex
+++ b/lib/music_library_web/components/core_components.ex
@@ -489,7 +489,7 @@ defmodule MusicLibraryWeb.CoreComponents do
         <tbody
           id={@id}
           phx-update={match?(%Phoenix.LiveView.LiveStream{}, @rows) && "stream"}
-          class="relative divide-y divide-zinc-100 border-t border-zinc-200 text-sm leading-6 text-zinc-700"
+          class="relative divide-y divide-zinc-100 border-t border-b border-zinc-200 text-sm leading-6 text-zinc-700"
         >
           <tr :for={row <- @rows} id={@row_id && @row_id.(row)} class="group hover:bg-zinc-50">
             <td
diff --git a/lib/music_library_web/components/pagination.ex b/lib/music_library_web/components/pagination.ex
index 67b56ade..6c446678 100644
--- a/lib/music_library_web/components/pagination.ex
+++ b/lib/music_library_web/components/pagination.ex
@@ -11,10 +11,7 @@ defmodule MusicLibraryWeb.Pagination do
     assigns = assign(assigns, :page_links, page_links)
 
     ~H"""
-    <div
-      id={@id}
-      class="flex items-center justify-between border-t border-zinc-200 bg-white px-4 py-6 sm:px-6 mb-4"
-    >
+    <div id={@id} class="flex items-center justify-between bg-white px-4 py-6 mb-4">
       <%!-- Only on smallest viewport --%>
       <div class={[
         "flex flex-1 sm:hidden",
diff --git a/lib/music_library_web/live/record_live/index.html.heex b/lib/music_library_web/live/record_live/index.html.heex
index 92b770e9..f308335b 100644
--- a/lib/music_library_web/live/record_live/index.html.heex
+++ b/lib/music_library_web/live/record_live/index.html.heex
@@ -1,30 +1,32 @@
-<header class="gap-6 mb-2">
-  <h1 class="text-lg font-semibold leading-8 text-zinc-800">All Records</h1>
-  <div class="flex items-center justify-between gap-6 mb-2 mt-2">
-    <form class="w-2/3" for={@record_list_params.query} phx-submit="search" phx-change="search">
-      <input
-        type="search"
-        class="w-full rounded-lg text-zinc-900 border-gray-300 focus:border-zinc-400 focus:ring-0 sm:text-sm sm:leading-6"
-        id={:query}
-        name={:query}
-        value={@record_list_params.query}
-        placeholder="Search"
-        phx-debounce="500"
-        autocorrect="off"
-        autocapitalize="none"
-      />
-    </form>
-    <.link patch={~p"/records/import"}>
-      <.button>Import</.button>
-    </.link>
-  </div>
-</header>
+<div class="px-4 sm:px-0">
+  <header class="gap-6 mb-2">
+    <h1 class="text-lg font-semibold leading-8 text-zinc-800">All Records</h1>
+    <div class="flex items-center justify-between gap-6 mb-2 mt-2">
+      <form class="w-2/3" for={@record_list_params.query} phx-submit="search" phx-change="search">
+        <input
+          type="search"
+          class="w-full rounded-lg text-zinc-900 border-gray-300 focus:border-zinc-400 focus:ring-0 sm:text-sm sm:leading-6"
+          id={:query}
+          name={:query}
+          value={@record_list_params.query}
+          placeholder="Search"
+          phx-debounce="500"
+          autocorrect="off"
+          autocapitalize="none"
+        />
+      </form>
+      <.link patch={~p"/records/import"}>
+        <.button>Import</.button>
+      </.link>
+    </div>
+  </header>
 
-<p class="text-right text-base max-sm:text-xs mt-8">
-  Showing <b><%= Enum.count(@streams.records) %></b>
-  of <b><%= @record_list_params.total_entries %></b>
-  records
-</p>
+  <p class="text-right text-base max-sm:text-xs mt-8">
+    Showing <b><%= Enum.count(@streams.records) %></b>
+    of <b><%= @record_list_params.total_entries %></b>
+    records
+  </p>
+</div>
 
 <.table
   id="records"