Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(insights): allow tooltip to be passed to transaction column headers #82588

Merged
merged 8 commits into from
Jan 3, 2025
Merged
14 changes: 7 additions & 7 deletions static/app/views/insights/pages/ai/aiOverviewPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -47,13 +47,13 @@ import {
} from 'sentry/views/performance/utils';

export const AI_COLUMN_TITLES = [
'transaction',
'operation',
'project',
'tpm',
'p50()',
'p95()',
'users',
{title: 'transaction'},
{title: 'operation'},
{title: 'project'},
{title: 'tpm'},
{title: 'p50()'},
{title: 'p95()'},
{title: 'users'},
];

function AiOverviewPage() {
Expand Down
22 changes: 11 additions & 11 deletions static/app/views/insights/pages/backend/backendOverviewPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -47,17 +47,17 @@ import {
} from 'sentry/views/performance/utils';

export const BACKEND_COLUMN_TITLES = [
'http method',
'transaction',
'operation',
'project',
'tpm',
'p50()',
'p95()',
'failure rate',
'apdex',
'users',
'user misery',
{title: 'http method'},
{title: 'transaction'},
{title: 'operation'},
{title: 'project'},
{title: 'tpm'},
{title: 'p50()'},
{title: 'p95()'},
{title: 'failure rate'},
{title: 'apdex'},
{title: 'users'},
{title: 'user misery'},
];

function BackendOverviewPage() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,13 +3,15 @@ import styled from '@emotion/styled';
import Feature from 'sentry/components/acl/feature';
import {COL_WIDTH_UNDEFINED} from 'sentry/components/gridEditable';
import * as Layout from 'sentry/components/layouts/thirds';
import ExternalLink from 'sentry/components/links/externalLink';
import {NoAccess} from 'sentry/components/noAccess';
import {DatePageFilter} from 'sentry/components/organizations/datePageFilter';
import {EnvironmentPageFilter} from 'sentry/components/organizations/environmentPageFilter';
import PageFilterBar from 'sentry/components/organizations/pageFilterBar';
import {ProjectPageFilter} from 'sentry/components/organizations/projectPageFilter';
import TransactionNameSearchBar from 'sentry/components/performance/searchBar';
import * as TeamKeyTransactionManager from 'sentry/components/performance/teamKeyTransactionsManager';
import {tct} from 'sentry/locale';
import {trackAnalytics} from 'sentry/utils/analytics';
import {
canUseMetricsData,
Expand Down Expand Up @@ -47,16 +49,25 @@ import {
ProjectPerformanceType,
} from 'sentry/views/performance/utils';

const DURATION_TOOLTIP = tct(
'A heuristic measuring when a pageload or navigation completes. Based on whether the initial load of the webpage has become idle [link:Learn more.]',
{
link: (
<ExternalLink href="https://docs.sentry.io/platforms/javascript/tracing/instrumentation/automatic-instrumentation/#idletimeout" />
),
}
);

export const FRONTEND_COLUMN_TITLES = [
'transaction',
'operation',
'project',
'tpm',
'p50()',
'p75()',
'p95()',
'users',
'user misery',
{title: 'transaction'},
{title: 'operation'},
{title: 'project'},
{title: 'tpm'},
{title: 'p50()', tooltip: DURATION_TOOLTIP},
{title: 'p75()', tooltip: DURATION_TOOLTIP},
{title: 'p95()', tooltip: DURATION_TOOLTIP},
{title: 'users'},
{title: 'user misery'},
];

function FrontendOverviewPage() {
Expand Down
40 changes: 22 additions & 18 deletions static/app/views/insights/pages/mobile/mobileOverviewPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -51,26 +51,26 @@ import {
} from 'sentry/views/performance/utils';

const MOBILE_COLUMN_TITLES = [
'transaction',
'operation',
'project',
'tpm',
'slow frame %',
'frozen frame %',
'users',
'user misery',
{title: 'transaction'},
{title: 'operation'},
{title: 'project'},
{title: 'tpm'},
{title: 'slow frame %'},
{title: 'frozen frame %'},
{title: 'users'},
{title: 'user misery'},
];

const REACT_NATIVE_COLUMN_TITLES = [
'transaction',
'operation',
'project',
'tpm',
'slow frame %',
'frozen frame %',
'stall %',
'users',
'user misery',
{title: 'transaction'},
{title: 'operation'},
{title: 'project'},
{title: 'tpm'},
{title: 'slow frame %'},
{title: 'frozen frame %'},
{title: 'stall %'},
{title: 'users'},
{title: 'user misery'},
];

function MobileOverviewPage() {
Expand Down Expand Up @@ -124,7 +124,11 @@ function MobileOverviewPage() {
);

if (organization.features.includes('mobile-vitals')) {
columnTitles = [...columnTitles.slice(0, 5), 'ttid', ...columnTitles.slice(5, 0)];
columnTitles = [
...columnTitles.slice(0, 5),
{title: 'ttid'},
...columnTitles.slice(5, 0),
];
tripleChartRowCharts.push(
...[
PerformanceWidgetSetting.TIME_TO_INITIAL_DISPLAY,
Expand Down
92 changes: 46 additions & 46 deletions static/app/views/performance/landing/data.tsx
Original file line number Diff line number Diff line change
@@ -1,60 +1,60 @@
export const FRONTEND_PAGELOAD_COLUMN_TITLES = [
'transaction',
'project',
'tpm',
'FCP',
'LCP',
'FID',
'CLS',
'users',
'user misery',
{title: 'transaction'},
{title: 'project'},
{title: 'tpm'},
{title: 'FCP'},
{title: 'LCP'},
{title: 'FID'},
{title: 'CLS'},
{title: 'users'},
{title: 'user misery'},
];

export const FRONTEND_OTHER_COLUMN_TITLES = [
'transaction',
'operation',
'project',
'tpm',
'p50()',
'p75()',
'p95()',
'users',
'user misery',
{title: 'transaction'},
{title: 'operation'},
{title: 'project'},
{title: 'tpm'},
{title: 'p50()'},
{title: 'p75()'},
{title: 'p95()'},
{title: 'users'},
{title: 'user misery'},
];

export const BACKEND_COLUMN_TITLES = [
'http method',
'transaction',
'operation',
'project',
'tpm',
'p50',
'p95',
'failure rate',
'apdex',
'users',
'user misery',
{title: 'http method'},
{title: 'transaction'},
{title: 'operation'},
{title: 'project'},
{title: 'tpm'},
{title: 'p50'},
{title: 'p95'},
{title: 'failure rate'},
{title: 'apdex'},
{title: 'users'},
{title: 'user misery'},
];

export const MOBILE_COLUMN_TITLES = [
'transaction',
'operation',
'project',
'tpm',
'slow frame %',
'frozen frame %',
'users',
'user misery',
{title: 'transaction'},
{title: 'operation'},
{title: 'project'},
{title: 'tpm'},
{title: 'slow frame %'},
{title: 'frozen frame %'},
{title: 'users'},
{title: 'user misery'},
];

export const REACT_NATIVE_COLUMN_TITLES = [
'transaction',
'operation',
'project',
'tpm',
'slow frame %',
'frozen frame %',
'stall %',
'users',
'user misery',
{title: 'transaction'},
{title: 'operation'},
{title: 'project'},
{title: 'tpm'},
{title: 'slow frame %'},
{title: 'frozen frame %'},
{title: 'stall %'},
{title: 'users'},
{title: 'user misery'},
];
2 changes: 1 addition & 1 deletion static/app/views/performance/landing/index.spec.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -198,7 +198,7 @@ describe('Performance > Landing > Index', function () {

expect(columnHeaders).toHaveLength(REACT_NATIVE_COLUMN_TITLES.length);
for (const [index, title] of columnHeaders.entries()) {
expect(title).toHaveTextContent(REACT_NATIVE_COLUMN_TITLES[index]!);
expect(title).toHaveTextContent(REACT_NATIVE_COLUMN_TITLES[index].title!);
}
});

Expand Down
6 changes: 5 additions & 1 deletion static/app/views/performance/landing/views/mobileView.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,11 @@ export function MobileView(props: BasePerformanceViewProps) {
];

if (organization.features.includes('mobile-vitals')) {
columnTitles = [...columnTitles.slice(0, 5), 'ttid', ...columnTitles.slice(5, 0)];
columnTitles = [
...columnTitles.slice(0, 5),
{title: 'ttid'},
...columnTitles.slice(5, 0),
];
allowedCharts.push(
...[
PerformanceWidgetSetting.TIME_TO_INITIAL_DISPLAY,
Expand Down
44 changes: 37 additions & 7 deletions static/app/views/performance/table.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import {Component, useEffect} from 'react';
import {Component, type ReactNode, useEffect} from 'react';
import styled from '@emotion/styled';
import type {Location, LocationDescriptorObject} from 'history';

Expand All @@ -11,9 +11,11 @@ import SortLink from 'sentry/components/gridEditable/sortLink';
import Link from 'sentry/components/links/link';
import LoadingIndicator from 'sentry/components/loadingIndicator';
import Pagination from 'sentry/components/pagination';
import QuestionTooltip from 'sentry/components/questionTooltip';
import {Tooltip} from 'sentry/components/tooltip';
import {IconStar} from 'sentry/icons';
import {t, tct} from 'sentry/locale';
import {space} from 'sentry/styles/space';
import type {Organization} from 'sentry/types/organization';
import type {Project} from 'sentry/types/project';
import {trackAnalytics} from 'sentry/utils/analytics';
Expand Down Expand Up @@ -55,14 +57,23 @@ import {
UNPARAMETERIZED_TRANSACTION,
} from './utils';

type ColumnTitle = {
title: string | ReactNode;
tooltip?: string | ReactNode;
};
bcoe marked this conversation as resolved.
Show resolved Hide resolved

const COLUMN_TITLES_OPTIONAL_TOOLTIP = COLUMN_TITLES.map(title => {
return {title};
});

type Props = {
eventView: EventView;
location: Location;
organization: Organization;
projects: Project[];
setError: (msg: string | undefined) => void;
withStaticFilters: boolean;
columnTitles?: string[];
columnTitles?: ColumnTitle[];
domainViewFilters?: DomainViewFilters;
summaryConditions?: string;
};
Expand Down Expand Up @@ -402,7 +413,7 @@ class _Table extends Component<Props, State> {
renderHeadCell(
tableMeta: TableData['meta'],
column: TableColumn<keyof TableDataRow>,
title: React.ReactNode
title: ColumnTitle
): React.ReactNode {
const {eventView, location} = this.props;

Expand Down Expand Up @@ -437,7 +448,7 @@ class _Table extends Component<Props, State> {
const sortLink = (
<SortLink
align={align}
title={title || field.field}
title={title.title || field.field}
direction={currentSortKind}
canSort={canSort}
generateSortLink={generateSortLink}
Expand All @@ -451,11 +462,20 @@ class _Table extends Component<Props, State> {
</GuideAnchor>
);
}
return sortLink;

if (!title.tooltip) {
return sortLink;
}
return (
<Header>
{sortLink}
<QuestionTooltip size="xs" position="top" title={title.tooltip} isHoverable />
</Header>
);
}

renderHeadCellWithMeta = (tableMeta: TableData['meta']) => {
const columnTitles = this.props.columnTitles ?? COLUMN_TITLES;
const columnTitles = this.props.columnTitles ?? COLUMN_TITLES_OPTIONAL_TOOLTIP;
return (column: TableColumn<keyof TableDataRow>, index: number): React.ReactNode =>
this.renderHeadCell(tableMeta, column, columnTitles[index]);
};
Expand All @@ -479,7 +499,9 @@ class _Table extends Component<Props, State> {
/>
</TeamKeyTransactionWrapper>
);
return [this.renderHeadCell(tableData?.meta, teamKeyTransactionColumn, star)];
return [
this.renderHeadCell(tableData?.meta, teamKeyTransactionColumn, {title: star}),
];
}
return [this.renderBodyCell(tableData, teamKeyTransactionColumn, dataRow)];
}
Expand Down Expand Up @@ -627,4 +649,12 @@ const UnparameterizedTooltipWrapper = styled('div')`
justify-content: center;
`;

const Header = styled('div')`
display: grid;
grid-template-columns: repeat(2, max-content);
align-items: center;
padding: ${space(1.5)};
grid-column-gap: ${space(0.5)};
`;

export default Table;
Loading