From 08a9709ecfe87e51fb999fd19c58ea4d5ad74e70 Mon Sep 17 00:00:00 2001
From: Cyrus Goh <hello@cyrusgoh.com>
Date: Fri, 6 Dec 2024 09:09:39 -0800
Subject: [PATCH] feedback

---
 .../projects/environment-variables/ActionsCell.svelte         | 2 +-
 .../[project]/-/settings/environment-variables/+page.svelte   | 4 ++--
 .../src/components/dropdown-menu/DropdownMenuContent.svelte   | 2 +-
 web-common/src/components/table/BasicTable.svelte             | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/web-admin/src/features/projects/environment-variables/ActionsCell.svelte b/web-admin/src/features/projects/environment-variables/ActionsCell.svelte
index 2a2cf6b5ddb..c875613b19f 100644
--- a/web-admin/src/features/projects/environment-variables/ActionsCell.svelte
+++ b/web-admin/src/features/projects/environment-variables/ActionsCell.svelte
@@ -24,7 +24,7 @@
       <ThreeDot size="16px" />
     </IconButton>
   </DropdownMenu.Trigger>
-  <DropdownMenu.Content align="start">
+  <DropdownMenu.Content align="start" class="min-w-[95px]">
     <DropdownMenu.Item
       class="font-normal flex items-center"
       on:click={() => {
diff --git a/web-admin/src/routes/[organization]/[project]/-/settings/environment-variables/+page.svelte b/web-admin/src/routes/[organization]/[project]/-/settings/environment-variables/+page.svelte
index f209f81687b..c5ebd549283 100644
--- a/web-admin/src/routes/[organization]/[project]/-/settings/environment-variables/+page.svelte
+++ b/web-admin/src/routes/[organization]/[project]/-/settings/environment-variables/+page.svelte
@@ -102,7 +102,7 @@
       <div class="flex flex-col gap-6 w-full">
         <div class="flex flex-col">
           <RadixLarge>Environment variables</RadixLarge>
-          <p class="text-base font-normal text-slate-700">
+          <p class="text-sm text-slate-700 font-medium">
             Manage your environment variables here. <a
               href="https://docs.rilldata.com/tutorials/administration/project/credentials-env-variable-management"
               target="_blank"
@@ -123,7 +123,7 @@
           />
           <DropdownMenu.Root>
             <DropdownMenu.Trigger
-              class="min-w-fit flex flex-row gap-1 items-center rounded-sm {isDropdownOpen
+              class="min-w-fit flex flex-row gap-1 items-center rounded-sm border border-slate-300 {isDropdownOpen
                 ? 'bg-slate-200'
                 : 'hover:bg-slate-100'} px-2 py-1"
             >
diff --git a/web-common/src/components/dropdown-menu/DropdownMenuContent.svelte b/web-common/src/components/dropdown-menu/DropdownMenuContent.svelte
index 398a892e840..5a8a675c916 100644
--- a/web-common/src/components/dropdown-menu/DropdownMenuContent.svelte
+++ b/web-common/src/components/dropdown-menu/DropdownMenuContent.svelte
@@ -17,7 +17,7 @@
   {transitionConfig}
   {sideOffset}
   class={cn(
-    "z-50 min-w-[8rem] rounded-[2px] border bg-popover p-1 text-popover-foreground shadow-md focus:outline-none",
+    "z-50 min-w-[8rem] rounded-[2px] border bg-popover p-1.5 text-popover-foreground shadow-md focus:outline-none",
     className,
   )}
   {...$$restProps}
diff --git a/web-common/src/components/table/BasicTable.svelte b/web-common/src/components/table/BasicTable.svelte
index 664606bbe23..9c7056df558 100644
--- a/web-common/src/components/table/BasicTable.svelte
+++ b/web-common/src/components/table/BasicTable.svelte
@@ -81,7 +81,7 @@
                   style={`margin-left: ${marginLeft};`}
                   class:cursor-pointer={header.column.getCanSort()}
                   class:select-none={header.column.getCanSort()}
-                  class="font-semibold text-gray-500 flex flex-row items-center gap-x-1 truncate"
+                  class="font-semibold text-gray-500 flex flex-row items-center gap-x-1 truncate text-sm"
                 >
                   <svelte:component
                     this={flexRender(