From 0ae27ad9edbdd7b624db2a6ba61734f99dd5496f Mon Sep 17 00:00:00 2001 From: hsiegeln <37154749+hsiegeln@users.noreply.github.com> Date: Thu, 23 Apr 2026 16:15:29 +0200 Subject: [PATCH] ui(drawer): reorder tabs Config first, default to Config --- .../CheckpointDetailDrawer/CheckpointDetailDrawer.test.tsx | 6 +++--- .../AppDeploymentPage/CheckpointDetailDrawer/index.tsx | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/ui/src/pages/AppsTab/AppDeploymentPage/CheckpointDetailDrawer/CheckpointDetailDrawer.test.tsx b/ui/src/pages/AppsTab/AppDeploymentPage/CheckpointDetailDrawer/CheckpointDetailDrawer.test.tsx index 17d8be95..a448a75c 100644 --- a/ui/src/pages/AppsTab/AppDeploymentPage/CheckpointDetailDrawer/CheckpointDetailDrawer.test.tsx +++ b/ui/src/pages/AppsTab/AppDeploymentPage/CheckpointDetailDrawer/CheckpointDetailDrawer.test.tsx @@ -60,11 +60,11 @@ describe('CheckpointDetailDrawer', () => { expect(screen.getByText(/alice/)).toBeInTheDocument(); }); - it('Logs tab is selected by default', () => { + it('Config tab is selected by default', () => { renderDrawer(); // Tabs from DS may render as buttons or tabs role — be lenient on the query - const logsTab = screen.getByText(/^logs$/i); - expect(logsTab).toBeInTheDocument(); + const configTab = screen.getByText(/^config$/i); + expect(configTab).toBeInTheDocument(); }); it('disables Restore when JAR is pruned', () => { diff --git a/ui/src/pages/AppsTab/AppDeploymentPage/CheckpointDetailDrawer/index.tsx b/ui/src/pages/AppsTab/AppDeploymentPage/CheckpointDetailDrawer/index.tsx index e82b7f96..b931674f 100644 --- a/ui/src/pages/AppsTab/AppDeploymentPage/CheckpointDetailDrawer/index.tsx +++ b/ui/src/pages/AppsTab/AppDeploymentPage/CheckpointDetailDrawer/index.tsx @@ -24,7 +24,7 @@ type TabId = 'logs' | 'config'; export function CheckpointDetailDrawer({ open, onClose, deployment, version, appSlug, envSlug, onRestore, currentForm, }: Props) { - const [tab, setTab] = useState('logs'); + const [tab, setTab] = useState('config'); const archived = !version; const title = ( @@ -67,8 +67,8 @@ export function CheckpointDetailDrawer({ active={tab} onChange={(t) => setTab(t as TabId)} tabs={[ - { value: 'logs', label: 'Logs' }, { value: 'config', label: 'Config' }, + { value: 'logs', label: 'Logs' }, ]} />