[eric] windows: settings and marketplace default to browser size, chrome-only naming, top separator line gone

This commit is contained in:
ciregenz
2026-08-05 14:20:25 -07:00
parent 0d07edf8c9
commit 07a05bd4d3
5 changed files with 14 additions and 26 deletions
@@ -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',
}}
>
@@ -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 }} />)}
@@ -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',
}}
>
@@ -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}
@@ -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;