Extract top albums component

lib/music_library_web/components/stats_components.ex

@@ -66,68 +66,6 @@ defmodule MusicLibraryWeb.StatsComponents do
     """
   end
 
-  attr :albums, :list, required: true
-  attr :collected_releases, :list, required: true
-  attr :wishlisted_releases, :list, required: true
-
-  def top_albums_by_period(assigns) do
-    ~H"""
-    <div class="mt-4">
-      <div class="space-y-2">
-        <div
-          :for={album <- @albums}
-          phx-click={
-            navigate_to_record(@collected_releases, @wishlisted_releases, album.album_musicbrainz_id)
-          }
-          class={[
-            "flex items-center space-x-3 p-2",
-            tracked_record?(@collected_releases ++ @wishlisted_releases, album.album_musicbrainz_id) &&
-              "cursor-pointer hover:bg-zinc-50 dark:hover:bg-zinc-800"
-          ]}
-        >
-          <img
-            class="w-12 h-12 rounded-md object-cover"
-            src={album.cover_url}
-            alt={album.album_title}
-          />
-          <div class="flex-1 min-w-0">
-            <.link
-              class="text-xs text-zinc-700 hover:text-zinc-500 dark:text-zinc-400 dark:hover:text-zinc-300 truncate"
-              navigate={~p"/artists/#{album.artist_musicbrainz_id}"}
-            >
-              {album.artist_name}
-            </.link>
-            <p class="text-sm font-medium text-zinc-900 dark:text-zinc-300 truncate">
-              {album.album_title}
-            </p>
-          </div>
-          <.badge :if={album.album_musicbrainz_id == ""}>
-            {album.play_count}
-          </.badge>
-          <.badge :if={
-            album.album_musicbrainz_id !== "" and
-              !tracked_record?(
-                @collected_releases ++ @wishlisted_releases,
-                album.album_musicbrainz_id
-              )
-          }>
-            {album.play_count}
-          </.badge>
-          <.badge :if={tracked_record?(@collected_releases, album.album_musicbrainz_id)} color="green">
-            {album.play_count}
-          </.badge>
-          <.badge
-            :if={tracked_record?(@wishlisted_releases, album.album_musicbrainz_id)}
-            color="yellow"
-          >
-            {album.play_count}
-          </.badge>
-        </div>
-      </div>
-    </div>
-    """
-  end
-
   attr :artists, :list, required: true
 
   def top_artists_by_period(assigns) do
@@ -191,17 +129,4 @@ defmodule MusicLibraryWeb.StatsComponents do
       if tracked_release.release_id == release_id, do: tracked_release.record_id
     end)
   end
-
-  defp navigate_to_record(collected_releases, wishlisted_releases, musicbrainz_id) do
-    cond do
-      record_id = tracked_record?(collected_releases, musicbrainz_id) ->
-        JS.navigate(~p"/collection/#{record_id}")
-
-      record_id = tracked_record?(wishlisted_releases, musicbrainz_id) ->
-        JS.navigate(~p"/wishlist/#{record_id}")
-
-      true ->
-        nil
-    end
-  end
 end

lib/music_library_web/live/stats_live/index.ex

@@ -6,6 +6,7 @@ defmodule MusicLibraryWeb.StatsLive.Index do
   import MusicLibraryWeb.StatsComponents
 
   alias MusicLibrary.{Collection, Records, ScrobbleActivity, Wishlist}
+  alias MusicLibraryWeb.StatsLive.TopAlbums
 
   def mount(_params, _session, socket) do
     latest_record = Collection.get_latest_record!()
@@ -28,7 +29,6 @@ defmodule MusicLibraryWeb.StatsLive.Index do
      )
      |> assign_counts()
      |> assign_scrobble_activity(recent_tracks)
-     |> assign_top_albums()
      |> assign_top_artists()
      |> assign(
        scrobble_activity_mode: "albums",
@@ -87,7 +87,6 @@ defmodule MusicLibraryWeb.StatsLive.Index do
 
     {:noreply,
      socket
-     |> assign_top_albums()
      |> assign_top_artists()
      |> assign_scrobble_activity(recent_tracks)}
   end
@@ -129,22 +128,6 @@ defmodule MusicLibraryWeb.StatsLive.Index do
     )
   end
 
-  defp assign_top_albums(socket) do
-    timezone = socket.assigns.timezone
-    current_time = DateTime.utc_now()
-
-    assign_async(socket, :top_albums, fn ->
-      top_albums =
-        ScrobbleActivity.get_top_albums_by_periods(
-          limit: 10,
-          current_time: current_time,
-          timezone: timezone
-        )
-
-      {:ok, %{top_albums: top_albums}}
-    end)
-  end
-
   defp assign_top_artists(socket) do
     timezone = socket.assigns.timezone
     current_time = DateTime.utc_now()

lib/music_library_web/live/stats_live/index.html.heex

@@ -102,54 +102,7 @@
         </.tabs>
       </.async_result>
     </div>
-    <div>
-      <h1 class="text-base lg:text-2xl text-zinc-900 dark:text-zinc-200 font-semibold">
-        {gettext("Top Albums")}
-      </h1>
-      <.async_result :let={top_albums} assign={@top_albums}>
-        <:loading>
-          <div class="h-192 flex items-center justify-center">
-            <.loading />
-          </div>
-        </:loading>
-        <.tabs class="mt-4">
-          <.tabs_list active_tab="top_albums_last_30_days" variant="segmented">
-            <:tab class="flex-1" name="top_albums_last_30_days">{gettext("Last 30 days")}</:tab>
-            <:tab class="flex-1" name="top_albums_last_90_days">{gettext("Last 90 days")}</:tab>
-            <:tab class="flex-1" name="top_albums_last_365_days">{gettext("Last year")}</:tab>
-            <:tab class="flex-1" name="top_albums_all_time">{gettext("All time")}</:tab>
-          </.tabs_list>
-          <.tabs_panel active name="top_albums_last_30_days">
-            <.top_albums_by_period
-              albums={top_albums.last_30_days}
-              collected_releases={top_albums.collected_releases}
-              wishlisted_releases={top_albums.wishlisted_releases}
-            />
-          </.tabs_panel>
-          <.tabs_panel name="top_albums_last_90_days">
-            <.top_albums_by_period
-              albums={top_albums.last_90_days}
-              collected_releases={top_albums.collected_releases}
-              wishlisted_releases={top_albums.wishlisted_releases}
-            />
-          </.tabs_panel>
-          <.tabs_panel name="top_albums_last_365_days">
-            <.top_albums_by_period
-              albums={top_albums.last_365_days}
-              collected_releases={top_albums.collected_releases}
-              wishlisted_releases={top_albums.wishlisted_releases}
-            />
-          </.tabs_panel>
-          <.tabs_panel name="top_albums_all_time">
-            <.top_albums_by_period
-              albums={top_albums.all_time}
-              collected_releases={top_albums.collected_releases}
-              wishlisted_releases={top_albums.wishlisted_releases}
-            />
-          </.tabs_panel>
-        </.tabs>
-      </.async_result>
-    </div>
+    <TopAlbums.live id="top-albums" timezone={@timezone} />
   </div>
 </div>
 

lib/music_library_web/live/stats_live/top_albums.ex

@@ -0,0 +1,166 @@
+defmodule MusicLibraryWeb.StatsLive.TopAlbums do
+  use MusicLibraryWeb, :live_component
+
+  import MusicLibraryWeb.StatsComponents, only: [tracked_record?: 2]
+
+  alias MusicLibrary.ScrobbleActivity
+
+  def live(assigns) do
+    ~H"""
+    <.live_component module={__MODULE__} {assigns} id={@id} />
+    """
+  end
+
+  @impl true
+  def render(assigns) do
+    ~H"""
+    <div>
+      <h1 class="text-base lg:text-2xl text-zinc-900 dark:text-zinc-200 font-semibold">
+        {gettext("Top Albums")}
+      </h1>
+      <.async_result :let={top_albums} assign={@top_albums}>
+        <:loading>
+          <div class="h-192 flex items-center justify-center">
+            <.loading />
+          </div>
+        </:loading>
+        <.tabs class="mt-4">
+          <.tabs_list active_tab="top_albums_last_30_days" variant="segmented">
+            <:tab class="flex-1" name="top_albums_last_30_days">{gettext("Last 30 days")}</:tab>
+            <:tab class="flex-1" name="top_albums_last_90_days">{gettext("Last 90 days")}</:tab>
+            <:tab class="flex-1" name="top_albums_last_365_days">{gettext("Last year")}</:tab>
+            <:tab class="flex-1" name="top_albums_all_time">{gettext("All time")}</:tab>
+          </.tabs_list>
+          <.tabs_panel active name="top_albums_last_30_days">
+            <.top_albums_by_period
+              albums={top_albums.last_30_days}
+              collected_releases={top_albums.collected_releases}
+              wishlisted_releases={top_albums.wishlisted_releases}
+            />
+          </.tabs_panel>
+          <.tabs_panel name="top_albums_last_90_days">
+            <.top_albums_by_period
+              albums={top_albums.last_90_days}
+              collected_releases={top_albums.collected_releases}
+              wishlisted_releases={top_albums.wishlisted_releases}
+            />
+          </.tabs_panel>
+          <.tabs_panel name="top_albums_last_365_days">
+            <.top_albums_by_period
+              albums={top_albums.last_365_days}
+              collected_releases={top_albums.collected_releases}
+              wishlisted_releases={top_albums.wishlisted_releases}
+            />
+          </.tabs_panel>
+          <.tabs_panel name="top_albums_all_time">
+            <.top_albums_by_period
+              albums={top_albums.all_time}
+              collected_releases={top_albums.collected_releases}
+              wishlisted_releases={top_albums.wishlisted_releases}
+            />
+          </.tabs_panel>
+        </.tabs>
+      </.async_result>
+    </div>
+    """
+  end
+
+  @impl true
+  def update(assigns, socket) do
+    {:ok,
+     socket
+     |> assign(assigns)
+     |> assign_top_albums()}
+  end
+
+  attr :albums, :list, required: true
+  attr :collected_releases, :list, required: true
+  attr :wishlisted_releases, :list, required: true
+
+  defp top_albums_by_period(assigns) do
+    ~H"""
+    <div class="mt-4">
+      <div class="space-y-2">
+        <div
+          :for={album <- @albums}
+          phx-click={
+            navigate_to_record(@collected_releases, @wishlisted_releases, album.album_musicbrainz_id)
+          }
+          class={[
+            "flex items-center space-x-3 p-2",
+            tracked_record?(@collected_releases ++ @wishlisted_releases, album.album_musicbrainz_id) &&
+              "cursor-pointer hover:bg-zinc-50 dark:hover:bg-zinc-800"
+          ]}
+        >
+          <img
+            class="w-12 h-12 rounded-md object-cover"
+            src={album.cover_url}
+            alt={album.album_title}
+          />
+          <div class="flex-1 min-w-0">
+            <.link
+              class="text-xs text-zinc-700 hover:text-zinc-500 dark:text-zinc-400 dark:hover:text-zinc-300 truncate"
+              navigate={~p"/artists/#{album.artist_musicbrainz_id}"}
+            >
+              {album.artist_name}
+            </.link>
+            <p class="text-sm font-medium text-zinc-900 dark:text-zinc-300 truncate">
+              {album.album_title}
+            </p>
+          </div>
+          <.badge :if={album.album_musicbrainz_id == ""}>
+            {album.play_count}
+          </.badge>
+          <.badge :if={
+            album.album_musicbrainz_id !== "" and
+              !tracked_record?(
+                @collected_releases ++ @wishlisted_releases,
+                album.album_musicbrainz_id
+              )
+          }>
+            {album.play_count}
+          </.badge>
+          <.badge :if={tracked_record?(@collected_releases, album.album_musicbrainz_id)} color="green">
+            {album.play_count}
+          </.badge>
+          <.badge
+            :if={tracked_record?(@wishlisted_releases, album.album_musicbrainz_id)}
+            color="yellow"
+          >
+            {album.play_count}
+          </.badge>
+        </div>
+      </div>
+    </div>
+    """
+  end
+
+  defp assign_top_albums(socket) do
+    timezone = socket.assigns.timezone
+    current_time = DateTime.utc_now()
+
+    assign_async(socket, :top_albums, fn ->
+      top_albums =
+        ScrobbleActivity.get_top_albums_by_periods(
+          limit: 10,
+          current_time: current_time,
+          timezone: timezone
+        )
+
+      {:ok, %{top_albums: top_albums}}
+    end)
+  end
+
+  defp navigate_to_record(collected_releases, wishlisted_releases, musicbrainz_id) do
+    cond do
+      record_id = tracked_record?(collected_releases, musicbrainz_id) ->
+        JS.navigate(~p"/collection/#{record_id}")
+
+      record_id = tracked_record?(wishlisted_releases, musicbrainz_id) ->
+        JS.navigate(~p"/wishlist/#{record_id}")
+
+      true ->
+        nil
+    end
+  end
+end

priv/gettext/default.pot

@@ -894,21 +894,24 @@ msgid "Record deleted"
 msgstr ""
 
 #: lib/music_library_web/live/stats_live/index.html.heex
+#: lib/music_library_web/live/stats_live/top_albums.ex
 #, elixir-autogen, elixir-format
 msgid "Last 30 days"
 msgstr ""
 
 #: lib/music_library_web/live/stats_live/index.html.heex
+#: lib/music_library_web/live/stats_live/top_albums.ex
 #, elixir-autogen, elixir-format
 msgid "Last 90 days"
 msgstr ""
 
-#: lib/music_library_web/live/stats_live/index.html.heex
+#: lib/music_library_web/live/stats_live/top_albums.ex
 #, elixir-autogen, elixir-format
 msgid "Top Albums"
 msgstr ""
 
 #: lib/music_library_web/live/stats_live/index.html.heex
+#: lib/music_library_web/live/stats_live/top_albums.ex
 #, elixir-autogen, elixir-format
 msgid "Last year"
 msgstr ""
@@ -934,6 +937,7 @@ msgid "Top Artists"
 msgstr ""
 
 #: lib/music_library_web/live/stats_live/index.html.heex
+#: lib/music_library_web/live/stats_live/top_albums.ex
 #, elixir-autogen, elixir-format
 msgid "All time"
 msgstr ""

priv/gettext/en/LC_MESSAGES/default.po

@@ -894,21 +894,24 @@ msgid "Record deleted"
 msgstr ""
 
 #: lib/music_library_web/live/stats_live/index.html.heex
+#: lib/music_library_web/live/stats_live/top_albums.ex
 #, elixir-autogen, elixir-format
 msgid "Last 30 days"
 msgstr ""
 
 #: lib/music_library_web/live/stats_live/index.html.heex
+#: lib/music_library_web/live/stats_live/top_albums.ex
 #, elixir-autogen, elixir-format
 msgid "Last 90 days"
 msgstr ""
 
-#: lib/music_library_web/live/stats_live/index.html.heex
+#: lib/music_library_web/live/stats_live/top_albums.ex
 #, elixir-autogen, elixir-format, fuzzy
 msgid "Top Albums"
 msgstr ""
 
 #: lib/music_library_web/live/stats_live/index.html.heex
+#: lib/music_library_web/live/stats_live/top_albums.ex
 #, elixir-autogen, elixir-format, fuzzy
 msgid "Last year"
 msgstr ""
@@ -934,6 +937,7 @@ msgid "Top Artists"
 msgstr ""
 
 #: lib/music_library_web/live/stats_live/index.html.heex
+#: lib/music_library_web/live/stats_live/top_albums.ex
 #, elixir-autogen, elixir-format
 msgid "All time"
 msgstr ""