Skip to content

Commit 2b0065d

Browse files
committed
ref(onboarding): Remove step indicators and animated logo from SCM flow
Remove "Step X of Y" text, "Optional" tag, and animated Sentry logo progress indicator from the SCM onboarding steps per design review feedback. Simplify ScmStepHeader to only render heading and subtitle.
1 parent 40cdd23 commit 2b0065d

File tree

5 files changed

+17
-63
lines changed

5 files changed

+17
-63
lines changed
Lines changed: 16 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -1,47 +1,27 @@
1-
import {Tag} from '@sentry/scraps/badge';
2-
import {Flex, Stack} from '@sentry/scraps/layout';
1+
import {Stack} from '@sentry/scraps/layout';
32
import {Heading, Text} from '@sentry/scraps/text';
43

5-
import {t} from 'sentry/locale';
6-
74
interface ScmStepHeaderProps {
85
heading: string;
9-
stepNumber: number;
106
subtitle: string;
11-
tag?: string;
12-
totalSteps?: number;
137
}
148

15-
export function ScmStepHeader({
16-
stepNumber,
17-
totalSteps = 3,
18-
heading,
19-
subtitle,
20-
tag,
21-
}: ScmStepHeaderProps) {
9+
export function ScmStepHeader({heading, subtitle}: ScmStepHeaderProps) {
2210
return (
23-
<Stack align="center" gap="md">
24-
<Flex align="center" gap="lg">
25-
<Text variant="muted" size="lg" bold density="comfortable">
26-
{t('Step %s of %s', stepNumber, totalSteps)}
27-
</Text>
28-
{tag && <Tag variant="muted">{tag}</Tag>}
29-
</Flex>
30-
<Stack align="center" gap="sm">
31-
<Heading as="h2" size="3xl">
32-
{heading}
33-
</Heading>
34-
<Text
35-
variant="muted"
36-
size="lg"
37-
wrap="pre-line"
38-
align="center"
39-
bold
40-
density="comfortable"
41-
>
42-
{subtitle}
43-
</Text>
44-
</Stack>
11+
<Stack align="center" gap="sm">
12+
<Heading as="h2" size="3xl">
13+
{heading}
14+
</Heading>
15+
<Text
16+
variant="muted"
17+
size="lg"
18+
wrap="pre-line"
19+
align="center"
20+
bold
21+
density="comfortable"
22+
>
23+
{subtitle}
24+
</Text>
4525
</Stack>
4626
);
4727
}

static/app/views/onboarding/onboarding.tsx

Lines changed: 1 addition & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,9 @@ import styled from '@emotion/styled';
33
import {AnimatePresence, motion} from 'framer-motion';
44

55
import {Button} from '@sentry/scraps/button';
6-
import {Container, Stack} from '@sentry/scraps/layout';
6+
import {Stack} from '@sentry/scraps/layout';
77
import {Link} from '@sentry/scraps/link';
88

9-
import {AnimatedSentryLogo} from 'sentry/components/animatedSentryLogo';
109
import Hook from 'sentry/components/hook';
1110
import {LogoSentry} from 'sentry/components/logoSentry';
1211
import {
@@ -102,16 +101,6 @@ const scmOnboardingSteps: StepDescriptor[] = [
102101
},
103102
];
104103

105-
/**
106-
* The SCM steps that display the animated logo progress indicator.
107-
* Order determines the progress level (first = 0, last = 1).
108-
*/
109-
const SCM_LOGO_STEPS = [
110-
OnboardingStepId.SCM_CONNECT,
111-
OnboardingStepId.SCM_PLATFORM_FEATURES,
112-
OnboardingStepId.SCM_PROJECT_DETAILS,
113-
];
114-
115104
function WelcomeVariable(props: StepProps) {
116105
const hasNewWelcomeUI = useHasNewWelcomeUI();
117106

@@ -318,12 +307,6 @@ export function OnboardingWithoutContext() {
318307
);
319308
};
320309

321-
const scmLogoIndex = stepObj ? SCM_LOGO_STEPS.indexOf(stepObj.id) : -1;
322-
const scmLogoProgress =
323-
scmLogoIndex >= 0 && SCM_LOGO_STEPS.length > 1
324-
? scmLogoIndex / (SCM_LOGO_STEPS.length - 1)
325-
: null;
326-
327310
// Redirect to the first step if we end up in an invalid state
328311
const isInvalidDocsStep = stepId === 'setup-docs' && !projectSlug;
329312
if (!stepObj || stepIndex === -1 || isInvalidDocsStep) {
@@ -393,11 +376,6 @@ export function OnboardingWithoutContext() {
393376
</Button>
394377
</BackMotionDiv>
395378
)}
396-
{scmLogoProgress !== null && (
397-
<Container alignSelf="center">
398-
<AnimatedSentryLogo progress={scmLogoProgress} />
399-
</Container>
400-
)}
401379
<AnimatePresence mode="wait" onExitComplete={updateAnimationState}>
402380
<OnboardingStepVariable id={stepObj.id} hasNewWelcomeUI={hasNewWelcomeUI}>
403381
{stepObj.Component && (

static/app/views/onboarding/scmConnect.tsx

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -63,10 +63,8 @@ export function ScmConnect({onComplete}: StepProps) {
6363
return (
6464
<Flex direction="column" align="center" gap="2xl" flexGrow={1}>
6565
<ScmStepHeader
66-
stepNumber={1}
6766
heading={t('Connect a repository')}
6867
subtitle={t('Link your source control for enhanced debugging features')}
69-
tag={t('Optional')}
7068
/>
7169

7270
<LayoutGroup>

static/app/views/onboarding/scmPlatformFeatures.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -372,7 +372,6 @@ export function ScmPlatformFeatures({onComplete}: StepProps) {
372372
return (
373373
<Flex direction="column" align="center" gap="3xl" flexGrow={1}>
374374
<ScmStepHeader
375-
stepNumber={2}
376375
heading={t('Platform & features')}
377376
subtitle={t('Select your SDK first, then choose the features to enable.')}
378377
/>

static/app/views/onboarding/scmProjectDetails.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,6 @@ export function ScmProjectDetails({onComplete}: StepProps) {
133133
return (
134134
<Flex direction="column" align="center" gap="2xl" flexGrow={1}>
135135
<ScmStepHeader
136-
stepNumber={3}
137136
heading={t('Project details')}
138137
subtitle={t(
139138
'Set the project name, assign a team, and configure\nhow you want to receive issue alerts'

0 commit comments

Comments
 (0)