✨ Applications: No tooltip when no selections

Signed-off-by: Gilles Dubreuil <gdubreui@redhat.com>
konveyor · Jul 10, 2023 · 10de0d2 · 10de0d2
1 parent 3a085a6
commit 10de0d2
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 24 deletions.
diff --git a/client/src/app/pages/applications/applications-table-analyze/applications-table-analyze.tsx b/client/src/app/pages/applications/applications-table-analyze/applications-table-analyze.tsx
@@ -455,24 +455,18 @@ export const ApplicationsTableAnalyze: React.FC = () => {
     ? [
         <ConditionalTooltip
           key="delete-app-tooltip"
-          isTooltipEnabled={
-            selectedRows.length < 1 ||
-            selectedRows.some(
-              (application) => application.migrationWave !== null
-            )
-          }
+          isTooltipEnabled={selectedRows.some(
+            (application) => application.migrationWave !== null
+          )}
           content={
             "Cannot delete application(s) assigned to migration wave(s)."
           }
         >
           <DropdownItem
             key="applications-bulk-delete"
-            isDisabled={
-              selectedRows.length < 1 ||
-              selectedRows.some(
-                (application) => application.migrationWave !== null
-              )
-            }
+            isDisabled={selectedRows.some(
+              (application) => application.migrationWave !== null
+            )}
             onClick={() => {
               openBulkDeleteModal(selectedRows);
             }}

diff --git a/...rc/app/pages/applications/applications-table-assessment/applications-table-assessment.tsx b/...rc/app/pages/applications/applications-table-assessment/applications-table-assessment.tsx
@@ -631,24 +631,18 @@ export const ApplicationsTable: React.FC = () => {
     ? [
         <ConditionalTooltip
           key="delete-app-tooltip"
-          isTooltipEnabled={
-            selectedRows.length < 1 ||
-            selectedRows.some(
-              (application) => application.migrationWave !== null
-            )
-          }
+          isTooltipEnabled={selectedRows.some(
+            (application) => application.migrationWave !== null
+          )}
           content={
             "Cannot delete application(s) assigned to migration wave(s)."
           }
         >
           <DropdownItem
             key="applications-bulk-delete"
-            isDisabled={
-              selectedRows.length < 1 ||
-              selectedRows.some(
-                (application) => application.migrationWave !== null
-              )
-            }
+            isDisabled={selectedRows.some(
+              (application) => application.migrationWave !== null
+            )}
             onClick={() => {
               openBulkDeleteModal(selectedRows);
             }}