Improve importing visual feedback

lib/music_library/records.ex

@@ -32,6 +32,12 @@ defmodule MusicLibrary.Records do
     Repo.one!(q)
   end
 
+  def search_release_group(query, opts \\ []) do
+    limit = Keyword.get(opts, :limit, 20)
+    offset = Keyword.get(opts, :offset, 0)
+    MusicBrainz.search_release_group(query, limit: limit, offset: offset)
+  end
+
   def import_from_musicbrainz(musicbrainz_id) do
     with {:ok, release_group} <- MusicBrainz.get_release_group(musicbrainz_id),
          {:ok, image_data} <- MusicBrainz.get_cover_art(musicbrainz_id),

lib/music_library/records/music_brainz.ex

@@ -148,9 +148,9 @@ defmodule MusicLibrary.Records.MusicBrainz do
         ]
       }
   """
-  def search_release_group(query, opts \\ []) do
-    limit = Keyword.get(opts, :limit, 20)
-    offset = Keyword.get(opts, :offset, 0)
+  def search_release_group(query, opts) do
+    limit = Keyword.fetch!(opts, :limit)
+    offset = Keyword.fetch!(opts, :offset)
 
     qs = [
       query: query,

lib/music_library_web/live/record_live/search_component.ex

@@ -1,7 +1,8 @@
 defmodule MusicLibraryWeb.RecordLive.SearchComponent do
   use MusicLibraryWeb, :live_component
 
-  alias MusicLibrary.Records.MusicBrainz
+  alias MusicLibrary.Records
+
   @impl true
   def render(assigns) do
     ~H"""
@@ -32,8 +33,7 @@ defmodule MusicLibraryWeb.RecordLive.SearchComponent do
     ~H"""
     <li
       class="flex justify-between gap-x-6 py-5 cursor-pointer hover:bg-gray-50"
-      phx-click={JS.push("import", value: %{id: @record.id})}
-      data-confirm="Are you sure you want to import this record?"
+      phx-click={JS.push("import", value: %{id: @record.id}, page_loading: true)}
     >
       <div class="flex min-w-0 gap-x-4">
         <div class="min-w-0 flex-auto">
@@ -85,7 +85,7 @@ defmodule MusicLibraryWeb.RecordLive.SearchComponent do
   defp search(""), do: {:ok, []}
 
   defp search(query) do
-    case MusicBrainz.search_release_group(query) do
+    case Records.search_release_group(query, limit: 10) do
       {:ok, result} ->
         {:ok,
          Enum.map(result["release-groups"], fn rg ->