Skip to content

Commit 7f8d6d9

Browse files
scttcperc298lee
authored andcommitted
fix(issues): Re-enable new user issues details guide (#77189)
1 parent e5b905f commit 7f8d6d9

File tree

1 file changed

+43
-37
lines changed

1 file changed

+43
-37
lines changed

static/app/components/events/breadcrumbs/breadcrumbsDataSection.tsx

Lines changed: 43 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import {useCallback, useMemo, useRef, useState} from 'react';
22
import styled from '@emotion/styled';
33

4+
import GuideAnchor from 'sentry/components/assistant/guideAnchor';
45
import {Button} from 'sentry/components/button';
56
import ButtonBar from 'sentry/components/buttonBar';
67
import ErrorBoundary from 'sentry/components/errorBoundary';
@@ -176,43 +177,48 @@ export default function BreadcrumbsDataSection({
176177
const hasViewAll = summaryCrumbs.length !== enhancedCrumbs.length;
177178

178179
return (
179-
<InterimSection
180-
key="breadcrumbs"
181-
type={SectionKey.BREADCRUMBS}
182-
title={t('Breadcrumbs')}
183-
data-test-id="breadcrumbs-data-section"
184-
actions={actions}
185-
>
186-
<ErrorBoundary mini message={t('There was an error loading the event breadcrumbs')}>
187-
<div ref={setContainer}>
188-
<BreadcrumbsTimeline
189-
breadcrumbs={summaryCrumbs}
190-
startTimeString={startTimeString}
191-
// We want the timeline to appear connected to the 'View All' button
192-
showLastLine={hasViewAll}
193-
fullyExpanded={false}
194-
containerElement={container}
195-
/>
196-
</div>
197-
{hasViewAll && (
198-
<ViewAllContainer>
199-
<VerticalEllipsis />
200-
<div>
201-
<ViewAllButton
202-
size="sm"
203-
// Since we've disabled the button as an 'outside click' for the drawer we can change
204-
// the operation based on the drawer state.
205-
onClick={() => (isDrawerOpen ? closeDrawer() : onViewAllBreadcrumbs())}
206-
aria-label={t('View All Breadcrumbs')}
207-
ref={viewAllButtonRef}
208-
>
209-
{t('View All')}
210-
</ViewAllButton>
211-
</div>
212-
</ViewAllContainer>
213-
)}
214-
</ErrorBoundary>
215-
</InterimSection>
180+
<GuideAnchor target="breadcrumbs" position="top">
181+
<InterimSection
182+
key="breadcrumbs"
183+
type={SectionKey.BREADCRUMBS}
184+
title={t('Breadcrumbs')}
185+
data-test-id="breadcrumbs-data-section"
186+
actions={actions}
187+
>
188+
<ErrorBoundary
189+
mini
190+
message={t('There was an error loading the event breadcrumbs')}
191+
>
192+
<div ref={setContainer}>
193+
<BreadcrumbsTimeline
194+
breadcrumbs={summaryCrumbs}
195+
startTimeString={startTimeString}
196+
// We want the timeline to appear connected to the 'View All' button
197+
showLastLine={hasViewAll}
198+
fullyExpanded={false}
199+
containerElement={container}
200+
/>
201+
</div>
202+
{hasViewAll && (
203+
<ViewAllContainer>
204+
<VerticalEllipsis />
205+
<div>
206+
<ViewAllButton
207+
size="sm"
208+
// Since we've disabled the button as an 'outside click' for the drawer we can change
209+
// the operation based on the drawer state.
210+
onClick={() => (isDrawerOpen ? closeDrawer() : onViewAllBreadcrumbs())}
211+
aria-label={t('View All Breadcrumbs')}
212+
ref={viewAllButtonRef}
213+
>
214+
{t('View All')}
215+
</ViewAllButton>
216+
</div>
217+
</ViewAllContainer>
218+
)}
219+
</ErrorBoundary>
220+
</InterimSection>
221+
</GuideAnchor>
216222
);
217223
}
218224

0 commit comments

Comments
 (0)