From c3db08fef836a4f13876fcc001e4693e7631eec3 Mon Sep 17 00:00:00 2001 From: Roshan Kumar Date: Tue, 3 Feb 2026 23:07:31 +0530 Subject: [PATCH 1/3] Align collection settings modal with Penpot design --- .../shared/settings/SettingsModal.tsx | 94 ++++++++++--------- src/renderer/components/ui/table.tsx | 2 +- 2 files changed, 53 insertions(+), 43 deletions(-) diff --git a/src/renderer/components/shared/settings/SettingsModal.tsx b/src/renderer/components/shared/settings/SettingsModal.tsx index 0afcf410..56429d38 100644 --- a/src/renderer/components/shared/settings/SettingsModal.tsx +++ b/src/renderer/components/shared/settings/SettingsModal.tsx @@ -67,39 +67,55 @@ export const SettingsModal = () => { - -
+ + {/* Header - Fixed */} -
+
- Collection Settings + + Collection Settings + + + + Variables + + + Environments + +
{/* Tabs - Takes remaining space */} - -
- - - Variables - - - Environments - - -
- -
- + +
+ +
+
- +
+
- + +
{ onEnvironmentSelect={setEditorSelectedEnvironment} onValidChange={setEnvironmentValid} /> - - - - {/* Footer - Fixed */} - -
- - +
+ +
-
-
+
+ ); diff --git a/src/renderer/components/ui/table.tsx b/src/renderer/components/ui/table.tsx index b8b8e824..26a6fd2a 100644 --- a/src/renderer/components/ui/table.tsx +++ b/src/renderer/components/ui/table.tsx @@ -60,7 +60,7 @@ const TableHead = React.forwardRef< Date: Fri, 6 Feb 2026 00:00:50 +0530 Subject: [PATCH 2/3] Refine collection settings modal styling and fix design token inconsistencies --- .../shared/settings/SettingsModal.tsx | 48 ++++++++----------- .../settings/VariableTab/VariableEditor.tsx | 16 ++++--- src/renderer/components/ui/table.tsx | 4 +- 3 files changed, 31 insertions(+), 37 deletions(-) diff --git a/src/renderer/components/shared/settings/SettingsModal.tsx b/src/renderer/components/shared/settings/SettingsModal.tsx index 56429d38..6f97b2f9 100644 --- a/src/renderer/components/shared/settings/SettingsModal.tsx +++ b/src/renderer/components/shared/settings/SettingsModal.tsx @@ -1,7 +1,6 @@ import { Dialog, DialogContent, - DialogFooter, DialogHeader, DialogTitle, DialogTrigger, @@ -67,25 +66,26 @@ export const SettingsModal = () => { - + {/* Header - Fixed */} -
+
- + Collection Settings - + Variables + Environments @@ -95,27 +95,17 @@ export const SettingsModal = () => { {/* Tabs - Takes remaining space */} -
+
-
- -
-
+
{ onEnvironmentSelect={setEditorSelectedEnvironment} onValidChange={setEnvironmentValid} /> -
- -
+
+ +
diff --git a/src/renderer/components/shared/settings/VariableTab/VariableEditor.tsx b/src/renderer/components/shared/settings/VariableTab/VariableEditor.tsx index a35abf69..57623adb 100644 --- a/src/renderer/components/shared/settings/VariableTab/VariableEditor.tsx +++ b/src/renderer/components/shared/settings/VariableTab/VariableEditor.tsx @@ -105,7 +105,7 @@ export const VariableEditor = memo( ( update(index, { value: e.target.value })} /> @@ -127,13 +130,14 @@ export const VariableEditor = memo( update(index, { description: e.target.value })} /> update(index, { secret: Boolean(checked) })} /> @@ -142,7 +146,7 @@ export const VariableEditor = memo(