From 38ee3a005ed64b6036e990681f3ad9de15227b21 Mon Sep 17 00:00:00 2001
From: Daniel Hougaard <daniel@infisical.com>
Date: Mon, 16 Sep 2024 22:26:36 +0400
Subject: [PATCH] Requested changes

---
 .../components/IntegrationAuditLogsSection.tsx       |  4 ++--
 .../ConfiguredIntegrationItem.tsx                    | 12 +++++++++---
 2 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/frontend/src/views/IntegrationsPage/IntegrationDetailsPage/components/IntegrationAuditLogsSection.tsx b/frontend/src/views/IntegrationsPage/IntegrationDetailsPage/components/IntegrationAuditLogsSection.tsx
index 29aa6a6df..c90f5fef7 100644
--- a/frontend/src/views/IntegrationsPage/IntegrationDetailsPage/components/IntegrationAuditLogsSection.tsx
+++ b/frontend/src/views/IntegrationsPage/IntegrationDetailsPage/components/IntegrationAuditLogsSection.tsx
@@ -21,7 +21,7 @@ export const IntegrationAuditLogsSection = ({ integration, orgId }: Props) => {
 
   // eslint-disable-next-line no-nested-ternary
   return subscription?.auditLogs ? (
-    <div className="h-full w-full rounded-lg border border-mineshaft-600 bg-mineshaft-900 p-4">
+    <div className="h-full w-full min-w-[51rem] rounded-lg border border-mineshaft-600 bg-mineshaft-900 p-4">
       <div className="mb-4 flex items-center justify-between border-b border-mineshaft-400 pb-4">
         <p className="text-lg font-semibold text-gray-200">Integration Logs</p>
         <p className="text-xs text-gray-400">
@@ -43,7 +43,7 @@ export const IntegrationAuditLogsSection = ({ integration, orgId }: Props) => {
       />
     </div>
   ) : !isLoading ? (
-    <div className="h-full w-full min-w-[50rem] rounded-lg border border-mineshaft-600 bg-mineshaft-900 p-4 opacity-60">
+    <div className="h-full w-full min-w-[51rem] rounded-lg border border-mineshaft-600 bg-mineshaft-900 p-4 opacity-60">
       <div className="mb-4 flex items-center justify-between border-b border-mineshaft-400 pb-4">
         <p className="text-lg font-semibold text-gray-200">Integration Logs</p>
       </div>
diff --git a/frontend/src/views/IntegrationsPage/components/IntegrationsSection/ConfiguredIntegrationItem.tsx b/frontend/src/views/IntegrationsPage/components/IntegrationsSection/ConfiguredIntegrationItem.tsx
index 919d6870f..4ae52a94e 100644
--- a/frontend/src/views/IntegrationsPage/components/IntegrationsSection/ConfiguredIntegrationItem.tsx
+++ b/frontend/src/views/IntegrationsPage/components/IntegrationsSection/ConfiguredIntegrationItem.tsx
@@ -35,8 +35,8 @@ export const ConfiguredIntegrationItem = ({
 
   return (
     <div
-      onClick={() => router.push(`/integrations/details/${integration.id}`)}
       className="max-w-8xl flex cursor-pointer justify-between rounded-md border border-mineshaft-600 bg-mineshaft-800 p-3 transition-all hover:bg-mineshaft-700"
+      onClick={() => router.push(`/integrations/details/${integration.id}`)}
       key={`integration-${integration?.id.toString()}`}
     >
       <div className="flex">
@@ -211,7 +211,10 @@ export const ConfiguredIntegrationItem = ({
         <div className="mr-1 flex items-end opacity-80 duration-200 hover:opacity-100">
           <Tooltip className="text-center" content="Manually sync integration secrets">
             <Button
-              onClick={() => onManualSyncIntegration()}
+              onClick={(e) => {
+                e.stopPropagation();
+                onManualSyncIntegration();
+              }}
               className="max-w-[2.5rem] border-none bg-mineshaft-500"
             >
               <FontAwesomeIcon icon={faRefresh} className="px-1 text-bunker-200" />
@@ -226,7 +229,10 @@ export const ConfiguredIntegrationItem = ({
             <div className="flex items-end opacity-80 duration-200 hover:opacity-100">
               <Tooltip content="Remove Integration">
                 <IconButton
-                  onClick={() => onRemoveIntegration()}
+                  onClick={(e) => {
+                    e.stopPropagation();
+                    onRemoveIntegration();
+                  }}
                   ariaLabel="delete"
                   isDisabled={!isAllowed}
                   colorSchema="danger"