Skip to content

Commit 594a67c

Browse files
fix(replay): Use ProjectAvatar over ProjectBadge (#69009)
No need to use badge when there's no label
1 parent 06f9689 commit 594a67c

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

static/app/components/replays/breadcrumbs/breadcrumbItem.tsx

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@ import {isValidElement, memo} from 'react';
33
import styled from '@emotion/styled';
44
import beautify from 'js-beautify';
55

6+
import ProjectAvatar from 'sentry/components/avatar/projectAvatar';
67
import {CodeSnippet} from 'sentry/components/codeSnippet';
7-
import ProjectBadge from 'sentry/components/idBadge/projectBadge';
88
import Link from 'sentry/components/links/link';
99
import ObjectInspector from 'sentry/components/objectInspector';
1010
import PanelItem from 'sentry/components/panels/panelItem';
@@ -154,22 +154,20 @@ function CrumbErrorIssue({frame}: {frame: FeedbackFrame | ErrorFrame}) {
154154
const project = useProjectFromSlug({organization, projectSlug: frame.data.projectSlug});
155155
const {groupId} = useReplayGroupContext();
156156

157-
const projectBadge = project ? (
158-
<ProjectBadge project={project} avatarSize={16} disableLink displayName={false} />
159-
) : null;
157+
const projectAvatar = project ? <ProjectAvatar project={project} size={16} /> : null;
160158

161159
if (String(frame.data.groupId) === groupId) {
162160
return (
163161
<CrumbIssueWrapper>
164-
{projectBadge}
162+
{projectAvatar}
165163
{frame.data.groupShortId}
166164
</CrumbIssueWrapper>
167165
);
168166
}
169167

170168
return (
171169
<CrumbIssueWrapper>
172-
{projectBadge}
170+
{projectAvatar}
173171
<Link
174172
to={
175173
isFeedbackFrame(frame)
@@ -189,6 +187,7 @@ function CrumbErrorIssue({frame}: {frame: FeedbackFrame | ErrorFrame}) {
189187
const CrumbIssueWrapper = styled('div')`
190188
display: flex;
191189
align-items: center;
190+
gap: ${space(0.5)};
192191
font-size: ${p => p.theme.fontSizeSmall};
193192
color: ${p => p.theme.subText};
194193
`;

0 commit comments

Comments
 (0)