From e28e363bd09117659b4f0d46d8918963ad098720 Mon Sep 17 00:00:00 2001
From: Pavel <nybkox@gmail.com>
Date: Tue, 19 Aug 2025 18:05:07 +0200
Subject: [PATCH] feat: keep tabs and table headers sticky, scroll table bodies

---
 packages/web/src/elements/TabControl.svelte   | 29 +++++++++++++++++--
 packages/web/src/tabs/ServerSummaryTab.svelte | 12 ++++++--
 .../web/src/widgets/SummaryDatabases.svelte   |  8 +++--
 .../web/src/widgets/SummaryProcesses.svelte   |  8 +++--
 .../web/src/widgets/SummaryVariables.svelte   |  8 +++--
 5 files changed, 50 insertions(+), 15 deletions(-)

diff --git a/packages/web/src/elements/TabControl.svelte b/packages/web/src/elements/TabControl.svelte
index cf7b8cd93..87cc6dff6 100644
--- a/packages/web/src/elements/TabControl.svelte
+++ b/packages/web/src/elements/TabControl.svelte
@@ -15,7 +15,10 @@
   export let menu = null;
   export let isInline = false;
   export let containerMaxWidth = undefined;
+  export let containerMaxHeight = undefined;
   export let flex1 = true;
+  export let flexColContainer = true;
+  export let maxHeight100 = false;
   export let contentTestId = undefined;
   export let inlineTabs = false;
   export let onUserChange = null;
@@ -28,7 +31,7 @@
   }
 </script>
 
-<div class="main" class:flex1>
+<div class="main" class:maxHeight100 class:flex1>
   <div class="tabs" class:inlineTabs>
     {#each _.compact(tabs) as tab, index}
       <div
@@ -50,9 +53,16 @@
     {/if}
   </div>
 
-  <div class="content-container" data-testid={contentTestId}>
+  <div class="content-container" style:max-height={containerMaxHeight} data-testid={contentTestId}>
     {#each _.compact(tabs) as tab, index}
-      <div class="container" class:isInline class:tabVisible={index == value} style:max-width={containerMaxWidth}>
+      <div
+        class="container"
+        class:flexColContainer
+        class:maxHeight100
+        class:isInline
+        class:tabVisible={index == value}
+        style:max-width={containerMaxWidth}
+      >
         <svelte:component
           this={tab.component}
           {...tab.props}
@@ -88,6 +98,10 @@
     max-width: 100%;
   }
 
+  .main.maxHeight100 {
+    max-height: 100%;
+  }
+
   .tabs {
     display: flex;
     height: var(--dim-tabs-height);
@@ -137,6 +151,15 @@
     position: relative;
   }
 
+  .container.maxHeight100 {
+    max-height: 100%;
+  }
+
+  .container.flexColContainer {
+    display: flex;
+    flex-direction: column;
+  }
+
   .container:not(.isInline) {
     position: absolute;
     display: flex;
diff --git a/packages/web/src/tabs/ServerSummaryTab.svelte b/packages/web/src/tabs/ServerSummaryTab.svelte
index e923e73d8..b43d2dec6 100644
--- a/packages/web/src/tabs/ServerSummaryTab.svelte
+++ b/packages/web/src/tabs/ServerSummaryTab.svelte
@@ -96,9 +96,12 @@
       <div class="wrapper">
         <TabControl
           isInline
-          inlineTabs={true}
+          inlineTabs
           containerMaxWidth="100%"
-          flex1={true}
+          containerMaxHeight="calc(100% - 34px)"
+          maxHeight100
+          flex1
+          flexColContainer
           value={$serverSummarySelectedTab}
           onUserChange={index => serverSummarySelectedTab.set(index)}
           tabs={[
@@ -140,7 +143,9 @@
     right: 0;
     bottom: 0;
     background-color: var(--theme-bg-0);
-    overflow: auto;
+    height: 100%;
+    display: flex;
+    flex-direction: column;
   }
 
   .action-separator {
@@ -151,6 +156,7 @@
     display: flex;
     align-items: center;
     justify-content: center;
+    flex: 1;
   }
 
   .error-message {
diff --git a/packages/web/src/widgets/SummaryDatabases.svelte b/packages/web/src/widgets/SummaryDatabases.svelte
index be29fd98f..ccd0b27c5 100644
--- a/packages/web/src/widgets/SummaryDatabases.svelte
+++ b/packages/web/src/widgets/SummaryDatabases.svelte
@@ -23,12 +23,14 @@
   }));
 </script>
 
-<div>
+<div class="wrapper">
   <TableControl {filters} stickyHeader {rows} columns={tableColumns} />
 </div>
 
 <style>
-  div {
-    padding: 10px;
+  .wrapper {
+    flex-grow: 1;
+    overflow-y: auto;
+    max-height: 100%;
   }
 </style>
diff --git a/packages/web/src/widgets/SummaryProcesses.svelte b/packages/web/src/widgets/SummaryProcesses.svelte
index 133134e14..9966f2b4b 100644
--- a/packages/web/src/widgets/SummaryProcesses.svelte
+++ b/packages/web/src/widgets/SummaryProcesses.svelte
@@ -80,7 +80,7 @@
   });
 </script>
 
-<div>
+<div class="wrapper">
   <TableControl
     {filters}
     stickyHeader
@@ -162,8 +162,10 @@
 </div>
 
 <style>
-  div {
-    padding: 10px;
+  .wrapper {
+    flex-grow: 1;
+    overflow-y: auto;
+    max-height: 100%;
   }
 
   code {
diff --git a/packages/web/src/widgets/SummaryVariables.svelte b/packages/web/src/widgets/SummaryVariables.svelte
index 553993cec..8f6bdfb25 100644
--- a/packages/web/src/widgets/SummaryVariables.svelte
+++ b/packages/web/src/widgets/SummaryVariables.svelte
@@ -8,7 +8,7 @@
   const filters = writable({});
 </script>
 
-<div>
+<div class="wrapper">
   <TableControl
     stickyHeader
     rows={variables}
@@ -36,7 +36,9 @@
 </div>
 
 <style>
-  div {
-    padding: 10px;
+  .wrapper {
+    flex-grow: 1;
+    overflow-y: auto;
+    max-height: 100%;
   }
 </style>