fixes setstate, and double tooltip

awx/ui_next/src/components/JobCancelButton/JobCancelButton.jsx

@@ -33,7 +33,7 @@ function JobCancelButton({
           <Button
             aria-label={title}
             ouiaId="cancel job"
-            onClick={setIsOpen}
+            onClick={() => setIsOpen(true)}
             variant="plain"
           >
             <MinusCircleIcon />
@@ -43,7 +43,7 @@ function JobCancelButton({
             aria-label={title}
             variant="secondary"
             ouiaId="cancel job"
-            onClick={setIsOpen}
+            onClick={() => setIsOpen(true)}
           >{t`Cancel Sync`}</Button>
         )}
       </Tooltip>
@@ -77,7 +77,7 @@ function JobCancelButton({
             </Button>,
           ]}
         >
-          {t`Are you sure you want to submit the request to cancel this job?`}
+          {t`Are you sure you want to cancel this job?`}
         </AlertModal>
       )}
       {error && (

awx/ui_next/src/screens/Inventory/InventorySources/InventorySourceListItem.jsx

@@ -92,11 +92,8 @@ function InventorySourceListItem({
         </Td>
         <Td dataLabel={t`Type`}>{label}</Td>
         <ActionsTd dataLabel={t`Actions`}>
-          <ActionItem
+          {['running', 'pending', 'waiting'].includes(source?.status) ? (
-            visible={source.summary_fields.user_capabilities.start}
+            <ActionItem visible={source.summary_fields.user_capabilities.start}>
-            tooltip={t`Sync`}
-          >
-            {['running', 'pending', 'waiting'].includes(source?.status) ? (
               <JobCancelButton
                 job={{
                   type: 'inventory_update',
@@ -107,10 +104,15 @@ function InventorySourceListItem({
                 title={t`Cancel Inventory Source Sync`}
                 showIconButton
               />
-            ) : (
+            </ActionItem>
+          ) : (
+            <ActionItem
+              visible={source.summary_fields.user_capabilities.start}
+              tooltip={t`Sync`}
+            >
               <InventorySourceSyncButton source={source} />
-            )}
+            </ActionItem>
-          </ActionItem>
+          )}
           <ActionItem
             visible={source.summary_fields.user_capabilities.edit}
             tooltip={t`Edit`}

awx/ui_next/src/screens/Project/ProjectList/ProjectListItem.jsx

@@ -169,11 +169,10 @@ function ProjectListItem({
           />
         </Td>
         <ActionsTd dataLabel={t`Actions`}>
-          <ActionItem
+          {['running', 'pending', 'waiting'].includes(job?.status) ? (
-            visible={project.summary_fields.user_capabilities.start}
+            <ActionItem
-            tooltip={t`Sync Project`}
+              visible={project.summary_fields.user_capabilities.start}
-          >
+            >
-            {['running', 'pending', 'waiting'].includes(job?.status) ? (
               <JobCancelButton
                 job={{ id: job.id, type: 'project_update' }}
                 errorTitle={t`Project Sync Error`}
@@ -181,13 +180,18 @@ function ProjectListItem({
                 showIconButton
                 errorMessage={t`Failed to cancel Project Sync`}
               />
-            ) : (
+            </ActionItem>
+          ) : (
+            <ActionItem
+              visible={project.summary_fields.user_capabilities.start}
+              tooltip={t`Sync Project`}
+            >
               <ProjectSyncButton
                 projectId={project.id}
                 lastJobStatus={job && job.status}
               />
-            )}
+            </ActionItem>
-          </ActionItem>
+          )}
           <ActionItem
             visible={project.summary_fields.user_capabilities.edit}
             tooltip={t`Edit Project`}