From 5aa6a7a8e709faed9ff0e402302dda02d864798d Mon Sep 17 00:00:00 2001
From: Abdenasser <nasser.elidrissi065@gmail.com>
Date: Wed, 13 Nov 2024 14:08:29 +0100
Subject: [PATCH] ProcessRow component

---
 src/lib/components/process/ProcessRow.svelte  | 81 +++++++++++++++
 .../components/process/ProcessTable.svelte    | 98 +++----------------
 .../process/cells/ProcessCell.svelte          |  0
 3 files changed, 94 insertions(+), 85 deletions(-)
 delete mode 100644 src/lib/components/process/cells/ProcessCell.svelte

diff --git a/src/lib/components/process/ProcessRow.svelte b/src/lib/components/process/ProcessRow.svelte
index e69de29..07ea55b 100644
--- a/src/lib/components/process/ProcessRow.svelte
+++ b/src/lib/components/process/ProcessRow.svelte
@@ -0,0 +1,81 @@
+<script lang="ts">
+  import type { Process, Column } from "$lib/types";
+  import ProcessIcon from "./ProcessIcon.svelte";
+  import ActionButtons from "./ActionButtons.svelte";
+
+  export let process: Process;
+  export let columns: Column[];
+  export let systemStats: { memory_total: number } | null;
+  export let isPinned: boolean;
+  export let isHighUsage: boolean;
+
+  export let onTogglePin: (command: string) => void;
+  export let onShowDetails: (process: Process) => void;
+  export let onKillProcess: (process: Process) => void;
+</script>
+
+<tr class:high-usage={isHighUsage} class:pinned={isPinned}>
+  {#each columns.filter((col) => col.visible) as column}
+    <td class="truncate">
+      {#if column.id === "name"}
+        <div class="name-cell">
+          <ProcessIcon processName={process.name} />
+          <span class="process-name">{process.name}</span>
+        </div>
+      {:else if column.format}
+        {@html column.format(process[column.id])}
+      {:else}
+        {process[column.id]}
+      {/if}
+    </td>
+  {/each}
+  <ActionButtons
+    {process}
+    {isPinned}
+    {onTogglePin}
+    {onShowDetails}
+    {onKillProcess}
+  />
+</tr>
+
+<style>
+  td {
+    padding: 6px 12px;
+    border-bottom: 1px solid var(--surface0);
+    color: var(--text);
+    z-index: 1;
+  }
+
+  .truncate {
+    white-space: nowrap;
+    overflow: hidden;
+    text-overflow: ellipsis;
+    max-width: 0;
+  }
+
+  tr:hover {
+    background-color: var(--surface0);
+  }
+
+  .high-usage {
+    background-color: color-mix(in srgb, var(--red) 10%, transparent);
+  }
+
+  .high-usage:hover {
+    background-color: color-mix(in srgb, var(--red) 15%, transparent);
+  }
+
+  tr.pinned {
+    background-color: color-mix(in srgb, var(--blue) 10%, transparent);
+  }
+
+  tr.pinned:hover {
+    background-color: color-mix(in srgb, var(--blue) 15%, transparent);
+  }
+
+  .name-cell {
+    display: flex;
+    align-items: center;
+    gap: 8px;
+  }
+</style>
diff --git a/src/lib/components/process/ProcessTable.svelte b/src/lib/components/process/ProcessTable.svelte
index b26327c..7291e9f 100644
--- a/src/lib/components/process/ProcessTable.svelte
+++ b/src/lib/components/process/ProcessTable.svelte
@@ -1,8 +1,7 @@
 <script lang="ts">
   import type { Process, Column } from "$lib/types";
-  import ProcessIcon from "./ProcessIcon.svelte";
   import TableHeader from "./TableHeader.svelte";
-  import ActionButtons from "./ActionButtons.svelte";
+  import ProcessRow from "./ProcessRow.svelte";
 
   export let processes: Process[];
   export let columns: Column[];
@@ -21,33 +20,17 @@
     <TableHeader {columns} {sortConfig} {onToggleSort} />
     <tbody>
       {#each processes as process (process.pid)}
-        <tr
-          class:high-usage={process.cpu_usage > 50 ||
+        <ProcessRow
+          {process}
+          {columns}
+          {systemStats}
+          isPinned={pinnedProcesses.has(process.command)}
+          isHighUsage={process.cpu_usage > 50 ||
             process.memory_usage / (systemStats?.memory_total || 0) > 0.1}
-          class:pinned={pinnedProcesses.has(process.command)}
-        >
-          {#each columns.filter((col) => col.visible) as column}
-            <td class="truncate">
-              {#if column.id === "name"}
-                <div class="name-cell">
-                  <ProcessIcon processName={process.name} />
-                  <span class="process-name">{process.name}</span>
-                </div>
-              {:else if column.format}
-                {@html column.format(process[column.id])}
-              {:else}
-                {process[column.id]}
-              {/if}
-            </td>
-          {/each}
-          <ActionButtons
-            {process}
-            isPinned={pinnedProcesses.has(process.command)}
-            {onTogglePin}
-            {onShowDetails}
-            {onKillProcess}
-          />
-        </tr>
+          {onTogglePin}
+          {onShowDetails}
+          {onKillProcess}
+        />
       {/each}
     </tbody>
   </table>
@@ -58,13 +41,10 @@
     flex: 1;
     overflow-x: auto;
     overflow-y: auto;
-
-    /* Scrollbar styles */
-    scrollbar-width: thin; /* Firefox */
-    scrollbar-color: var(--surface2) var(--mantle); /* Firefox */
+    scrollbar-width: thin;
+    scrollbar-color: var(--surface2) var(--mantle);
   }
 
-  /* Webkit scrollbar styles (Chrome, Safari, Edge) */
   .table-container::-webkit-scrollbar {
     width: 8px;
     height: 8px;
@@ -89,11 +69,6 @@
     background: var(--mantle);
   }
 
-  /* When both scrollbars are present, add some padding to prevent overlap */
-  .table-container::-webkit-scrollbar-corner {
-    background-color: var(--mantle);
-  }
-
   table {
     width: max-content;
     min-width: 100%;
@@ -101,51 +76,4 @@
     border-collapse: collapse;
     font-size: 13px;
   }
-
-  td {
-    padding: 6px 12px;
-    border-bottom: 1px solid var(--surface0);
-    color: var(--text);
-    z-index: 1;
-  }
-
-  tr:hover {
-    background-color: var(--surface0);
-  }
-
-  .truncate {
-    white-space: nowrap;
-    overflow: hidden;
-    text-overflow: ellipsis;
-    max-width: 0;
-  }
-
-  .high-usage {
-    background-color: color-mix(in srgb, var(--red) 10%, transparent);
-  }
-
-  .high-usage:hover {
-    background-color: color-mix(in srgb, var(--red) 15%, transparent);
-  }
-
-  tr.pinned {
-    background-color: color-mix(in srgb, var(--blue) 10%, transparent);
-  }
-
-  tr.pinned:hover {
-    background-color: color-mix(in srgb, var(--blue) 15%, transparent);
-  }
-
-  td:last-child {
-    width: 120px;
-    min-width: 120px;
-    max-width: 120px;
-    padding: 6px 8px;
-  }
-
-  .name-cell {
-    display: flex;
-    align-items: center;
-    gap: 8px;
-  }
 </style>
diff --git a/src/lib/components/process/cells/ProcessCell.svelte b/src/lib/components/process/cells/ProcessCell.svelte
deleted file mode 100644
index e69de29..0000000