From 558980a1917e5b92798f758e7414eca4ac12e449 Mon Sep 17 00:00:00 2001
From: Graham McIntire <graham@mcintire.me>
Date: Sat, 2 May 2026 17:02:56 -0500
Subject: [PATCH] polish(rover): drop redundant View action; location cell
 already links

---
 lib/microwaveprop_web/live/rover_locations_live.ex    | 10 +---------
 .../live/rover_locations_live_test.exs                | 11 ++---------
 2 files changed, 3 insertions(+), 18 deletions(-)

diff --git a/lib/microwaveprop_web/live/rover_locations_live.ex b/lib/microwaveprop_web/live/rover_locations_live.ex
index a7b3dbfa..c01a8c9a 100644
--- a/lib/microwaveprop_web/live/rover_locations_live.ex
+++ b/lib/microwaveprop_web/live/rover_locations_live.ex
@@ -277,15 +277,8 @@ defmodule MicrowavepropWeb.RoverLocationsLive do
     assigns = %{record: record, owner?: owner?(scope, record)}
 
     ~H"""
-    <div class="flex gap-2">
-      <.link
-        navigate={~p"/rover-locations/#{@record.id}"}
-        class="btn btn-xs btn-ghost"
-      >
-        View
-      </.link>
+    <div :if={@owner?} class="flex gap-2">
       <button
-        :if={@owner?}
         type="button"
         phx-click="edit"
         phx-value-id={@record.id}
@@ -294,7 +287,6 @@ defmodule MicrowavepropWeb.RoverLocationsLive do
         Edit
       </button>
       <button
-        :if={@owner?}
         type="button"
         phx-click="delete"
         phx-value-id={@record.id}
diff --git a/test/microwaveprop_web/live/rover_locations_live_test.exs b/test/microwaveprop_web/live/rover_locations_live_test.exs
index 1e363eda..409008fc 100644
--- a/test/microwaveprop_web/live/rover_locations_live_test.exs
+++ b/test/microwaveprop_web/live/rover_locations_live_test.exs
@@ -121,7 +121,7 @@ defmodule MicrowavepropWeb.RoverLocationsLiveTest do
       assert html =~ ~s(value="EM12gm")
     end
 
-    test "View link on each row navigates to the show page", %{conn: conn, user: user} do
+    test "the location cell links to the show page", %{conn: conn, user: user} do
       {:ok, loc} =
         Rover.create_location(user, %{
           lat: 32.5,
@@ -132,14 +132,7 @@ defmodule MicrowavepropWeb.RoverLocationsLiveTest do
 
       {:ok, lv, _html} = live(conn, ~p"/rover-locations")
 
-      assert {:ok, _show_lv, html} =
-               lv
-               |> element("a[href='/rover-locations/#{loc.id}']", "View")
-               |> render_click()
-               |> follow_redirect(conn, ~p"/rover-locations/#{loc.id}")
-
-      assert html =~ "rooftop spot"
-      assert html =~ "EM12gm"
+      assert has_element?(lv, "a[href='/rover-locations/#{loc.id}']")
     end
 
     test "edit/delete buttons appear only on the owner's row", %{conn: conn, user: user} do