From 07a05bd4d3e4bb1861e1c9d6678f336c8e5bd690 Mon Sep 17 00:00:00 2001
From: ciregenz <ciregenz@berkeley.edu>
Date: Wed, 5 Aug 2026 14:20:25 -0700
Subject: [PATCH] [eric] windows: settings and marketplace default to browser
 size, chrome-only naming, top separator line gone

---
 .../pages/Directory/MarketplaceAppCard.tsx    |  2 +-
 .../app/pages/Directory/MarketplaceBody.tsx   |  7 +------
 .../app/pages/Settings/SettingsAppCard.tsx    |  2 +-
 .../src/app/pages/Settings/SettingsBody.tsx   | 21 +++++++------------
 .../src/shared/state/dashboardLayoutSlice.ts  |  8 +++----
 5 files changed, 14 insertions(+), 26 deletions(-)

diff --git a/frontend/src/app/pages/Directory/MarketplaceAppCard.tsx b/frontend/src/app/pages/Directory/MarketplaceAppCard.tsx
index 06eb7ea2..5d092eee 100644
--- a/frontend/src/app/pages/Directory/MarketplaceAppCard.tsx
+++ b/frontend/src/app/pages/Directory/MarketplaceAppCard.tsx
@@ -94,7 +94,7 @@ const MarketplaceAppCard: React.FC<Props> = ({
             onLostPointerCapture={header.onLostPointerCapture}
             style={{
               height: 42, flex: 'none', display: 'flex', alignItems: 'center', gap: 14,
-              padding: '0 16px', borderBottom: `1px solid ${c.border.subtle}`, background: c.bg.surface,
+              padding: '0 16px', background: c.bg.surface,
               cursor: header.dragging ? 'grabbing' : 'grab', touchAction: 'none', userSelect: 'none',
             }}
           >
diff --git a/frontend/src/app/pages/Directory/MarketplaceBody.tsx b/frontend/src/app/pages/Directory/MarketplaceBody.tsx
index 480d9d77..cddff735 100644
--- a/frontend/src/app/pages/Directory/MarketplaceBody.tsx
+++ b/frontend/src/app/pages/Directory/MarketplaceBody.tsx
@@ -78,12 +78,7 @@ const MarketplaceBody: React.FC = () => {
 
   return (
     <Box sx={{ display: 'flex', flexDirection: 'column', flex: 1, minHeight: 0, bgcolor: c.bg.surface }}>
-      <Box sx={{ px: 3.5, pt: 2.5, pb: 1.5, flexShrink: 0 }}>
-        <Typography sx={{ fontSize: '1.75rem', fontWeight: 600, color: c.text.primary, fontFamily: 'Georgia, "Times New Roman", serif', lineHeight: 1.15 }}>
-          Marketplace
-        </Typography>
-      </Box>
-      <Box sx={{ display: 'flex', flex: 1, minHeight: 0 }}>
+      <Box sx={{ display: 'flex', flex: 1, minHeight: 0, pt: 1.5 }}>
         <Box sx={{ width: 210, minWidth: 210, px: 2, pt: 0.5, display: 'flex', flexDirection: 'column', gap: 0.5 }}>
           {railRow('skills', 'Skills', <DescriptionOutlinedIcon sx={{ fontSize: 19, color: c.text.secondary }} />)}
           {railRow('connectors', 'Connectors', <GridViewOutlinedIcon sx={{ fontSize: 19, color: c.text.secondary }} />)}
diff --git a/frontend/src/app/pages/Settings/SettingsAppCard.tsx b/frontend/src/app/pages/Settings/SettingsAppCard.tsx
index 4791acaf..483c47ce 100644
--- a/frontend/src/app/pages/Settings/SettingsAppCard.tsx
+++ b/frontend/src/app/pages/Settings/SettingsAppCard.tsx
@@ -96,7 +96,7 @@ const SettingsAppCard: React.FC<Props> = ({
             onLostPointerCapture={header.onLostPointerCapture}
             style={{
               height: 42, flex: 'none', display: 'flex', alignItems: 'center', gap: 14,
-              padding: '0 16px', borderBottom: `1px solid ${c.border.subtle}`, background: c.bg.surface,
+              padding: '0 16px', background: c.bg.surface,
               cursor: header.dragging ? 'grabbing' : 'grab', touchAction: 'none', userSelect: 'none',
             }}
           >
diff --git a/frontend/src/app/pages/Settings/SettingsBody.tsx b/frontend/src/app/pages/Settings/SettingsBody.tsx
index eed78e04..f57d3d26 100644
--- a/frontend/src/app/pages/Settings/SettingsBody.tsx
+++ b/frontend/src/app/pages/Settings/SettingsBody.tsx
@@ -86,24 +86,18 @@ const SettingsBody: React.FC<SettingsBodyProps> = ({ active, onRequestClose }) =
   const styles = makeSettingsStyles(c);
 
   return (
-    <Box sx={{ flex: 1, minHeight: 0, display: 'flex', flexDirection: 'column', overflow: 'hidden', bgcolor: c.bg.surface }}>
-      <Box sx={{ display: 'flex', alignItems: 'flex-start', justifyContent: 'space-between', px: 3.5, pt: 2.5, pb: 1.5, flexShrink: 0 }}>
-        <Typography sx={{ fontSize: '1.75rem', fontWeight: 600, color: c.text.primary, fontFamily: 'Georgia, "Times New Roman", serif', lineHeight: 1.15 }}>
-          Settings
+    <Box sx={{ flex: 1, minHeight: 0, display: 'flex', flexDirection: 'row', overflow: 'hidden', bgcolor: c.bg.surface, pt: 1.5 }}>
+      <SettingsRail activeTab={activeTab} onTabChange={(v) => setActiveTab(v as SettingsTab)} />
+
+      <Box sx={{ flex: 1, minWidth: 0, display: 'flex', flexDirection: 'column' }}>
+      <Box sx={{ display: 'flex', alignItems: 'center', justifyContent: 'space-between', px: 3, pt: 0.5, pb: 0.75, flexShrink: 0 }}>
+        <Typography sx={{ color: c.text.primary, fontWeight: 600, fontSize: '1rem' }}>
+          {railLabelFor(activeTab)}
         </Typography>
         <IconButton onClick={handleRequestClose} size="small" data-onboarding="settings-close-button" sx={{ color: c.text.tertiary, '&:hover': { color: c.text.primary } }}>
           <X size={18} />
         </IconButton>
       </Box>
-      <Box sx={{ display: 'flex', flexDirection: 'row', flex: 1, minHeight: 0 }}>
-      <SettingsRail activeTab={activeTab} onTabChange={(v) => setActiveTab(v as SettingsTab)} />
-
-      <Box sx={{ flex: 1, minWidth: 0, display: 'flex', flexDirection: 'column' }}>
-      <Box sx={{ display: 'flex', alignItems: 'center', px: 3, pt: 0.5, pb: 0.75, flexShrink: 0 }}>
-        <Typography sx={{ color: c.text.primary, fontWeight: 600, fontSize: '1rem' }}>
-          {railLabelFor(activeTab)}
-        </Typography>
-      </Box>
 
       <Box sx={{
         px: 3,
@@ -169,7 +163,6 @@ const SettingsBody: React.FC<SettingsBodyProps> = ({ active, onRequestClose }) =
       )}
       </Box>
       </Box>
-      </Box>
 
       <DirectoryBrowser
         open={browseOpen}
diff --git a/frontend/src/shared/state/dashboardLayoutSlice.ts b/frontend/src/shared/state/dashboardLayoutSlice.ts
index 00d3d034..442bdcef 100644
--- a/frontend/src/shared/state/dashboardLayoutSlice.ts
+++ b/frontend/src/shared/state/dashboardLayoutSlice.ts
@@ -25,13 +25,13 @@ export const DEFAULT_WORKFLOW_CARD_H = 520;
 // Open at the same default footprint as a browser/view card so it lands at a comfortable size automatically.
 export const DEFAULT_WORKFLOWS_HUB_W = DEFAULT_BROWSER_CARD_W;
 export const DEFAULT_WORKFLOWS_HUB_H = DEFAULT_BROWSER_CARD_H;
-export const DEFAULT_SETTINGS_CARD_W = 900;
-export const DEFAULT_SETTINGS_CARD_H = 640;
+export const DEFAULT_SETTINGS_CARD_W = DEFAULT_BROWSER_CARD_W;
+export const DEFAULT_SETTINGS_CARD_H = DEFAULT_BROWSER_CARD_H;
 // The two singleton windows have no card map to key off, so they own these fixed ids everywhere (selection, minimize, z-order).
 export const SETTINGS_CARD_ID = 'settings';
 export const MARKETPLACE_CARD_ID = 'marketplace';
-export const DEFAULT_MARKETPLACE_CARD_W = 1080;
-export const DEFAULT_MARKETPLACE_CARD_H = 720;
+export const DEFAULT_MARKETPLACE_CARD_W = DEFAULT_BROWSER_CARD_W;
+export const DEFAULT_MARKETPLACE_CARD_H = DEFAULT_BROWSER_CARD_H;
 export const WORKFLOWS_HUB_ID = 'workflows-hub';
 export const EXPANDED_CARD_MIN_H = 620;
 export const GRID_GAP = 24;