From 0ab404f81a83d8362afee7919ed74f0f72c0acf0 Mon Sep 17 00:00:00 2001
From: soup <soup@r4tio.dev>
Date: Mon, 25 Dec 2023 19:50:58 +0100
Subject: [PATCH] fix(filters): hide tooltip when actions are active (#1318)

---
 web/src/screens/filters/List.tsx | 55 +++++++++++++++++++-------------
 1 file changed, 33 insertions(+), 22 deletions(-)

diff --git a/web/src/screens/filters/List.tsx b/web/src/screens/filters/List.tsx
index 3bc737a..ee9185f 100644
--- a/web/src/screens/filters/List.tsx
+++ b/web/src/screens/filters/List.tsx
@@ -639,28 +639,39 @@ function FilterListItem({ filter, values, idx }: FilterListItemProps) {
             ) : filter.priority}
           </span>
           <span className="z-10 whitespace-nowrap text-xs font-medium text-gray-600 dark:text-gray-400">
-            <Tooltip
-              label={
-                <Link
-                  to={`${filter.id.toString()}/actions`}
-                  className="flex items-center cursor-pointer hover:text-black dark:hover:text-gray-300"
-                >
-                  <span className={filter.actions_count === 0 || filter.actions_enabled_count === 0 ? "text-red-500 hover:text-red-400 dark:hover:text-red-400" : ""}>
-        Actions: {filter.actions_enabled_count}/{filter.actions_count}
-                  </span>
-                </Link>
-              }
-            >
-              {filter.actions_count === 0 ? (
-                <>
-                  {"No actions defined. Set up actions to enable snatching."}
-                </>
-              ) : filter.actions_enabled_count === 0 ? (
-                <>
-                  {"You need to enable at least one action in the filter otherwise you will not get any snatches."}
-                </>
-              ) : null}
-            </Tooltip>
+            {filter.actions_count === 0 || filter.actions_enabled_count === 0 ? (
+              <Tooltip
+                label={
+                  <Link
+                    to={`${filter.id.toString()}/actions`}
+                    className="flex items-center cursor-pointer hover:text-black dark:hover:text-gray-300"
+                  >
+                    <span className={filter.actions_count === 0 || filter.actions_enabled_count === 0 ? "text-red-500 hover:text-red-400 dark:hover:text-red-400" : ""}>
+          Actions: {filter.actions_enabled_count}/{filter.actions_count}
+                    </span>
+                  </Link>
+                }
+              >
+                {filter.actions_count === 0 ? (
+                  <>
+                    {"No actions defined. Set up actions to enable snatching."}
+                  </>
+                ) : filter.actions_enabled_count === 0 ? (
+                  <>
+                    {"You need to enable at least one action in the filter otherwise you will not get any snatches."}
+                  </>
+                ) : null}
+              </Tooltip>
+            ) : (
+              <Link
+                to={`${filter.id.toString()}/actions`}
+                className="flex items-center cursor-pointer hover:text-black dark:hover:text-gray-300"
+              >
+                <span>
+          Actions: {filter.actions_enabled_count}/{filter.actions_count}
+                </span>
+              </Link>
+            )}
           </span>
         </div>
       </div>