Migrate layouts to Phoenix v1.8 (#3196)

Author: aleDsz

lib/livebook/teams/requests.ex

@@ -223,14 +223,14 @@ defmodule Livebook.Teams.Requests do
     params = %{access_token: access_token}
 
     fun = fn
-      _, %Req.TransportError{reason: :econnrefused} -> not valid_cache?
-      _, %Req.Response{status: status} when status in [408, 429, 500, 502, 503, 504] -> true
-      _, %Req.TransportError{reason: reason} when reason in [:timeout, :closed] -> true
+      _, %{reason: :econnrefused} -> not valid_cache?
+      _, %{status: status} when status in [408, 429, 500, 502, 503, 504] -> true
+      _, %{reason: reason} when reason in [:timeout, :closed] -> true
       _, _ -> false
     end
 
     case Req.get(req, url: "/api/v1/org/identity", params: params, retry: fun) do
-      {:error, %Req.TransportError{reason: :econnrefused}} -> :econnrefused
+      {:error, %{reason: :econnrefused}} -> :econnrefused
       otherwise -> handle_response(otherwise)
     end
   end

lib/livebook_web.ex

@@ -3,9 +3,7 @@ defmodule LivebookWeb do
 
   def controller do
     quote do
-      use Phoenix.Controller,
-        formats: [:html, :json],
-        layouts: [html: LivebookWeb.Layouts]
+      use Phoenix.Controller, formats: [:html, :json]
 
       import Plug.Conn
 
@@ -15,7 +13,7 @@ defmodule LivebookWeb do
 
   def live_view do
     quote do
-      use Phoenix.LiveView, layout: {LivebookWeb.Layouts, :live}
+      use Phoenix.LiveView
 
       unquote(html_helpers())
     end
@@ -64,6 +62,8 @@ defmodule LivebookWeb do
 
       # Shortcut for generating JS commands
       alias Phoenix.LiveView.JS
+      # Shortcut for layout components
+      alias LivebookWeb.Layouts
 
       # Routes generation with the ~p sigil
       unquote(verified_routes())

lib/livebook_web/components/core_components.ex

@@ -88,6 +88,45 @@ defmodule LivebookWeb.CoreComponents do
     """
   end
 
+  @doc """
+  Renders page title.
+
+  ## Examples
+
+      <.title text="Learn" />
+
+  """
+  attr :text, :string, default: nil
+  attr :back_navigate, :string, default: nil
+
+  slot :inner_block
+
+  def title(assigns) do
+    if assigns.text == nil and assigns.inner_block == [] do
+      raise ArgumentError, "should pass at least text attribute or an inner block"
+    end
+
+    ~H"""
+    <div class="relative">
+      <div
+        :if={@back_navigate}
+        class="hidden md:flex absolute top-0 bottom-0 left-0 transform -translate-x-full"
+      >
+        <.link navigate={@back_navigate}>
+          <.remix_icon icon="arrow-left-line" class="align-middle mr-2 text-2xl text-gray-800" />
+        </.link>
+      </div>
+      <h1 class="text-2xl text-gray-800 font-medium">
+        <%= if @inner_block != [] do %>
+          {render_slot(@inner_block)}
+        <% else %>
+          {@text}
+        <% end %>
+      </h1>
+    </div>
+    """
+  end
+
   @doc """
   Renders a message notice.