Skip to content

Commit 371c105

Browse files
committed
add more status tooltip messages, and remove other tooltips
1 parent b739ef5 commit 371c105

File tree

2 files changed

+27
-38
lines changed

2 files changed

+27
-38
lines changed

static/app/components/group/inboxBadges/statusBadge.tsx

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,30 +18,37 @@ export function getBadgeProperties(
1818
return {
1919
tagType: 'highlight',
2020
status: t('Resolved'),
21+
tooltip: t('This issue was marked as fixed.'),
2122
};
2223
}
2324
if (status === 'unresolved') {
2425
if (substatus === GroupSubstatus.REGRESSED) {
2526
return {
2627
tagType: 'highlight',
2728
status: t('Regressed'),
29+
tooltip: t('This issue was resolved then occurred again.'),
2830
};
2931
}
3032
if (substatus === GroupSubstatus.ESCALATING) {
3133
return {
3234
tagType: 'error',
3335
status: t('Escalating'),
36+
tooltip: t('This issue has exceeded its forecasted event volume.'),
3437
};
3538
}
3639
if (substatus === GroupSubstatus.NEW) {
3740
return {
3841
tagType: 'warning',
3942
status: t('New'),
43+
tooltip: t('This issue first occurred in the last 7 days.'),
4044
};
4145
}
4246
return {
4347
tagType: 'default',
4448
status: t('Ongoing'),
49+
tooltip: t(
50+
'This issue was created more than 7 days ago or has manually been marked as reviewed.'
51+
),
4552
};
4653
}
4754
if (status === 'ignored') {
@@ -50,10 +57,10 @@ export function getBadgeProperties(
5057
status: t('Archived'),
5158
tooltip:
5259
substatus === GroupSubstatus.ARCHIVED_FOREVER
53-
? t('Archived forever')
60+
? t('Archived forever.')
5461
: substatus === GroupSubstatus.ARCHIVED_UNTIL_ESCALATING
55-
? t('Archived until escalating')
56-
: t('Archived until condition met'),
62+
? t('Archived until escalating.')
63+
: t('Archived until condition met.'),
5764
};
5865
}
5966
return undefined;

static/app/views/issueDetails/streamline/header/header.tsx

Lines changed: 17 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -157,19 +157,12 @@ export default function StreamlinedGroupHeader({
157157
</StatLink>
158158
))}
159159
</StatTitle>
160-
<Tooltip
161-
title={t(
162-
'The main message summarizing the event. Typically the error message or log string that triggered this issue.'
163-
)}
164-
isHoverable
165-
>
166-
<EventMessage
167-
data={group}
168-
level={group.level}
169-
message={secondaryTitle}
170-
type={group.type}
171-
/>
172-
</Tooltip>
160+
<EventMessage
161+
data={group}
162+
level={group.level}
163+
message={secondaryTitle}
164+
type={group.type}
165+
/>
173166
{issueTypeConfig.eventAndUserCounts.enabled && (
174167
<Fragment>
175168
<StatCount value={eventCount} aria-label={t('Event count')} />
@@ -185,35 +178,24 @@ export default function StreamlinedGroupHeader({
185178
)}
186179
{statusProps?.status ? (
187180
<Fragment>
188-
<Tooltip title={statusProps?.tooltip}>
189-
<Subtext>
190-
<Tooltip
191-
isHoverable
192-
title={tct('The status of the issue. [link:Learn more]', {
193-
link: (
194-
<ExternalLink href="https://docs.sentry.io/product/issues/states-triage/" />
195-
),
196-
})}
197-
>
198-
{statusProps?.status}
199-
</Tooltip>
200-
</Subtext>
181+
<Tooltip
182+
isHoverable
183+
title={tct('[tooltip] [link:Learn more]', {
184+
tooltip: statusProps?.tooltip ?? '',
185+
link: (
186+
<ExternalLink href="https://docs.sentry.io/product/issues/states-triage/" />
187+
),
188+
})}
189+
>
190+
<Subtext>{statusProps?.status}</Subtext>
201191
</Tooltip>
202192
</Fragment>
203193
) : null}
204194
{subtitle && (
205195
<Fragment>
206196
<Divider />
207197
<Subtitle title={subtitle} isHoverable showOnlyOnOverflow delay={1000}>
208-
<Subtext>
209-
<Tooltip
210-
title={
211-
'Some extra context (function, route, query, etc.) to help you spot where the error happened.'
212-
}
213-
>
214-
{subtitle}
215-
</Tooltip>
216-
</Subtext>
198+
<Subtext>{subtitle}</Subtext>
217199
</Subtitle>
218200
</Fragment>
219201
)}

0 commit comments

Comments
 (0)