From 1bb03b52c92ad4d03f5e1c439c93fa141d19d829 Mon Sep 17 00:00:00 2001
From: Moritz <moritz.m@local-it.org>
Date: Mon, 22 Dec 2025 17:59:48 +0100
Subject: [PATCH] Fix accessibility issues: add tooltip for disabled delete
 button

---
 .../live/membership_fee_type_live/index.ex    | 51 ++++++++++---------
 1 file changed, 27 insertions(+), 24 deletions(-)

diff --git a/lib/mv_web/live/membership_fee_type_live/index.ex b/lib/mv_web/live/membership_fee_type_live/index.ex
index 5990a69..d0f9901 100644
--- a/lib/mv_web/live/membership_fee_type_live/index.ex
+++ b/lib/mv_web/live/membership_fee_type_live/index.ex
@@ -85,34 +85,37 @@ defmodule MvWeb.MembershipFeeTypeLive.Index do
         </:action>
 
         <:action :let={mft}>
+          <div
+            :if={get_member_count(mft, @member_counts) > 0}
+            class="tooltip tooltip-left"
+            data-tip={
+              gettext("Cannot delete - %{count} member(s) assigned",
+                count: get_member_count(mft, @member_counts)
+              )
+            }
+          >
+            <button
+              phx-click="delete"
+              phx-value-id={mft.id}
+              data-confirm={gettext("Are you sure?")}
+              class="btn btn-ghost btn-xs text-error opacity-50 cursor-not-allowed"
+              aria-label={
+                gettext("Cannot delete - %{count} member(s) assigned",
+                  count: get_member_count(mft, @member_counts)
+                )
+              }
+              disabled={true}
+            >
+              <.icon name="hero-trash" class="size-4" />
+            </button>
+          </div>
           <button
+            :if={get_member_count(mft, @member_counts) == 0}
             phx-click="delete"
             phx-value-id={mft.id}
             data-confirm={gettext("Are you sure?")}
-            class={[
-              "btn btn-ghost btn-xs",
-              if(get_member_count(mft, @member_counts) > 0,
-                do: "text-error opacity-50 cursor-not-allowed",
-                else: "text-error"
-              )
-            ]}
-            aria-label={
-              if get_member_count(mft, @member_counts) > 0,
-                do:
-                  gettext("Cannot delete - %{count} member(s) assigned",
-                    count: get_member_count(mft, @member_counts)
-                  ),
-                else: gettext("Delete membership fee type")
-            }
-            title={
-              if get_member_count(mft, @member_counts) > 0,
-                do:
-                  gettext("Cannot delete - %{count} member(s) assigned",
-                    count: get_member_count(mft, @member_counts)
-                  ),
-                else: gettext("Delete")
-            }
-            disabled={get_member_count(mft, @member_counts) > 0}
+            class="btn btn-ghost btn-xs text-error"
+            aria-label={gettext("Delete membership fee type")}
           >
             <.icon name="hero-trash" class="size-4" />
           </button>