feat: Making buttons better. Removing unnecessary code. #37

dwyl · Dec 9, 2022 · 558b07d · 558b07d
1 parent d32c90d
commit 558b07d
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 18 deletions.
diff --git a/lib/app_web/controllers/item_controller.ex b/lib/app_web/controllers/item_controller.ex
@@ -13,11 +13,6 @@ defmodule AppWeb.ItemController do
       %Item{}
     end
 
-    person_email = ""
-    if Map.has_key?(conn.assigns, :person) do
-      person_email = Map.get(conn.assigns.person, :email)
-    end
-
     case Map.has_key?(conn.assigns, :person) do
       false ->
         items = []

diff --git a/lib/app_web/controllers/item_html/index.html.heex b/lib/app_web/controllers/item_html/index.html.heex
@@ -105,23 +105,21 @@
       <% end %>
     </section>
     <div class="fixed right-0 mr-10">
-      <button
-      class="inline-block px-6 py-2.5 bg-[#fea5a5] text-white font-medium text-xs leading-tight uppercase rounded shadow-md hover:bg-[#f76e6e] hover:shadow-lg focus:bg-[#f76e6e] focus:shadow-lg focus:outline-none focus:ring-0 active:bg-[#a23434] active:shadow-lg transition duration-150 ease-in-out"
-    >
-        <a href="/logout">
-          Logout
-        </a>
-      </button>
+      <.link
+        class="inline-block px-6 py-2.5 bg-[#fea5a5] text-white font-medium text-xs leading-tight uppercase rounded shadow-md hover:bg-[#f76e6e] hover:shadow-lg focus:bg-[#f76e6e] focus:shadow-lg focus:outline-none focus:ring-0 active:bg-[#a23434] active:shadow-lg transition duration-150 ease-in-out"
+        href="/logout"
+      >
+        Logout
+      </.link>
     </div>
   <% else %>
     <div class="fixed h-screen ml-[25%] flex justify-center items-center">
-      <button
+      <.link
       class="max-h-[35px] inline-block px-6 py-2.5 bg-blue-300 text-white font-medium text-xs leading-tight uppercase rounded shadow-md hover:bg-blue-700 hover:shadow-lg focus:bg-blue-700 focus:shadow-lg focus:outline-none focus:ring-0 active:bg-blue-800 active:shadow-lg transition duration-150 ease-in-out"
-    >
-        <a href="/login">
-          Login
-        </a>
-      </button>
+        href="/login"
+      >
+        Login
+      </.link>
     </div>
   <% end %>
 </div>