Add some context and disable buttons when necesary

Author: mbuffa

assets/css/tile_map.css

@@ -114,14 +114,12 @@ div.toolbar {
   justify-content: left;
 }
 div.toolbar button {
+  cursor: pointer;
+}
+div.toolbar * {
   margin: 0.5rem;
   padding: 0.5rem;
   font-size: 1.4em;
-  cursor: pointer;
-}
-div.toolbar img.morpheus {
-  max-width: 300px;
-  margin: auto;
 }
 ul.entities-list {
   background-color: black;

config/config.exs

@@ -45,7 +45,8 @@ config :esbuild,
 # Configures Elixir's Logger
 config :logger, :console,
   format: "$time $metadata[$level] $message\n",
-  metadata: [:request_id]
+  metadata: [:request_id],
+  level: :info
 
 # Use Jason for JSON parsing in Phoenix
 config :phoenix, :json_library, Jason

lib/path_demo_web/live/map_live.ex

@@ -34,15 +34,14 @@ defmodule PathDemoWeb.MapLive do
   end
 
   def render(assigns) do
-    # <img class="morpheus" src="/images/morpheus.jpg">
-
     ~L"""
     <h3>Pathfinding A* Live Demo</h3>
 
     <div class="tilemap-container">
       <div class="toolbar">
-        <button phx-click="update-path">Find the path</button>
-        <button phx-click="walk-path">Walk the path</button>
+        <button phx-click="update-path" <%= update_disable(assigns)%>>Find the path</button>
+        <button phx-click="walk-path" <%= walk_disable(assigns) %>>Walk the path</button>
+        <p><%= if is_nil(assigns.player_target), do: "Pick a target!", else: "" %></p>
       </div>
 
       <ul class="entities-list">
@@ -225,4 +224,18 @@ defmodule PathDemoWeb.MapLive do
   def tile_in_closed_list?(%{closed_list: closed_list}, position) do
     closed_list |> Enum.any?(fn node -> node.position == position end)
   end
+
+  defp update_disable(%{path: %{final_path: final_path}, player_target: player_target} = _assigns) do
+    if is_nil(player_target) or not is_nil(final_path) do
+      "disabled"
+    else
+      ""
+    end
+  end
+  defp update_disable(%{player_target: nil}), do: "disabled"
+  defp update_disable(_), do: ""
+
+  defp walk_disable(%{path: %{final_path: nil}} = _assigns), do: "disabled"
+  defp walk_disable(%{path: nil} = _assigns), do: "disabled"
+  defp walk_disable(_), do: ""
 end