Skip to content

Commit dea4c6e

Browse files
JonasBashashjar
authored andcommitted
insights: use floating tabs (#102451)
Segmented controls should be used when we are toggling small widget options, not entire content areas - in this case, the tabs are more appropriate
1 parent 5ad5cb1 commit dea4c6e

File tree

1 file changed

+23
-23
lines changed

1 file changed

+23
-23
lines changed

static/app/views/insights/agents/views/overview.tsx

Lines changed: 23 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import {parseAsString, useQueryState} from 'nuqs';
44

55
import TransparentLoadingMask from 'sentry/components/charts/transparentLoadingMask';
66
import {Container, Flex, Stack} from 'sentry/components/core/layout';
7-
import {SegmentedControl} from 'sentry/components/core/segmentedControl';
7+
import {TabList, TabPanels, Tabs} from 'sentry/components/core/tabs';
88
import * as Layout from 'sentry/components/layouts/thirds';
99
import LoadingIndicator from 'sentry/components/loadingIndicator';
1010
import {DatePageFilter} from 'sentry/components/organizations/datePageFilter';
@@ -60,9 +60,6 @@ import {useSpans} from 'sentry/views/insights/common/queries/useDiscover';
6060
import {useDefaultToAllProjects} from 'sentry/views/insights/common/utils/useDefaultToAllProjects';
6161
import {ModuleName} from 'sentry/views/insights/types';
6262

63-
const TableControl = SegmentedControl<TableType>;
64-
const TableControlItem = SegmentedControl.Item<TableType>;
65-
6663
function useShowOnboarding() {
6764
const {projects} = useProjects();
6865
const pageFilters = usePageFilters();
@@ -220,26 +217,29 @@ function AgentsOverviewPage() {
220217
</WidgetGrid.Position3>
221218
</WidgetGrid>
222219
<Container paddingTop="xl" paddingBottom="xl">
223-
<TableControl
224-
value={activeTable}
225-
onChange={handleTableSwitch}
226-
size="sm"
227-
>
228-
<TableControlItem key={TableType.TRACES}>
229-
{t('Traces')}
230-
</TableControlItem>
231-
<TableControlItem key={TableType.MODELS}>
232-
{t('Models')}
233-
</TableControlItem>
234-
<TableControlItem key={TableType.TOOLS}>
235-
{t('Tools')}
236-
</TableControlItem>
237-
</TableControl>
220+
<Tabs value={activeTable} onChange={handleTableSwitch}>
221+
<TabList variant="floating">
222+
<TabList.Item key={TableType.TRACES}>
223+
{t('Traces')}
224+
</TabList.Item>
225+
<TabList.Item key={TableType.MODELS}>
226+
{t('Models')}
227+
</TabList.Item>
228+
<TabList.Item key={TableType.TOOLS}>{t('Tools')}</TabList.Item>
229+
</TabList>
230+
<TabPanels>
231+
<TabPanels.Item key={TableType.TRACES}>
232+
<TracesView />
233+
</TabPanels.Item>
234+
<TabPanels.Item key={TableType.MODELS}>
235+
<ModelsView />
236+
</TabPanels.Item>
237+
<TabPanels.Item key={TableType.TOOLS}>
238+
<ToolsView />
239+
</TabPanels.Item>
240+
</TabPanels>
241+
</Tabs>
238242
</Container>
239-
240-
{activeTable === TableType.TRACES && <TracesView />}
241-
{activeTable === TableType.MODELS && <ModelsView />}
242-
{activeTable === TableType.TOOLS && <ToolsView />}
243243
</Fragment>
244244
)}
245245
</ModuleLayout.Full>

0 commit comments

Comments
 (0)