Skip to content

Commit 8e28842

Browse files
committed
ref(onboarding): Replace legacySize icons with design system sizes
Use size="md" with accent/secondary variants for alert frequency and project details icons. Align section headers consistently.
1 parent bbc6024 commit 8e28842

File tree

2 files changed

+19
-13
lines changed

2 files changed

+19
-13
lines changed

static/app/views/onboarding/components/scmAlertFrequency.tsx

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ function AlertOptionCard({
6161
}
6262
}}
6363
>
64-
<Flex gap="md" align="start" style={{cursor: 'pointer'}}>
64+
<Flex gap="md" align="center">
6565
<Container padding="xs 0 0 0">
6666
<Radio size="sm" readOnly checked={isSelected} tabIndex={-1} />
6767
</Container>
@@ -91,21 +91,25 @@ export function ScmAlertFrequency({
9191
notificationProps,
9292
onFieldChange,
9393
}: ScmAlertFrequencyProps) {
94+
const isDefaultSelected = alertSetting === RuleAction.DEFAULT_ALERT;
9495
const isCustomSelected = alertSetting === RuleAction.CUSTOMIZED_ALERTS;
96+
const isLaterSelected = alertSetting === RuleAction.CREATE_ALERT_LATER;
9597

9698
return (
9799
<Stack gap="xl" role="radiogroup" aria-label={t('Alert frequency')}>
98100
<Stack gap="lg">
99101
<AlertOptionCard
100102
label={t('High priority issues')}
101-
icon={<IconWarning legacySize="16px" />}
102-
isSelected={alertSetting === RuleAction.DEFAULT_ALERT}
103+
icon={
104+
<IconWarning size="md" variant={isDefaultSelected ? 'accent' : 'secondary'} />
105+
}
106+
isSelected={isDefaultSelected}
103107
onSelect={() => onFieldChange('alertSetting', RuleAction.DEFAULT_ALERT)}
104108
/>
105109

106110
<AlertOptionCard
107111
label={t('Custom')}
108-
icon={<IconFix legacySize="16px" />}
112+
icon={<IconFix size="md" variant={isCustomSelected ? 'accent' : 'secondary'} />}
109113
isSelected={isCustomSelected}
110114
onSelect={() => onFieldChange('alertSetting', RuleAction.CUSTOMIZED_ALERTS)}
111115
>
@@ -179,8 +183,10 @@ export function ScmAlertFrequency({
179183

180184
<AlertOptionCard
181185
label={t("I'll create my own alerts later")}
182-
icon={<IconClock legacySize="16px" />}
183-
isSelected={alertSetting === RuleAction.CREATE_ALERT_LATER}
186+
icon={
187+
<IconClock size="md" variant={isLaterSelected ? 'accent' : 'secondary'} />
188+
}
189+
isSelected={isLaterSelected}
184190
onSelect={() => onFieldChange('alertSetting', RuleAction.CREATE_ALERT_LATER)}
185191
/>
186192
</Stack>

static/app/views/onboarding/scmProjectDetails.tsx

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -148,8 +148,8 @@ export function ScmProjectDetails({onComplete}: StepProps) {
148148
layout="position"
149149
>
150150
<Stack gap="sm">
151-
<Flex gap="md" align="center">
152-
<IconProject legacySize="16px" />
151+
<Flex gap="md" align="center" justify="center">
152+
<IconProject size="md" variant="secondary" />
153153
<Text bold size="lg" density="comfortable">
154154
{t('Give your project a name')}
155155
</Text>
@@ -171,8 +171,8 @@ export function ScmProjectDetails({onComplete}: StepProps) {
171171
</Stack>
172172

173173
<Stack gap="sm">
174-
<Flex gap="md" align="center">
175-
<IconGroup legacySize="16px" />
174+
<Flex gap="md" align="center" justify="center">
175+
<IconGroup size="md" />
176176
<Text bold size="lg" density="comfortable">
177177
{t('Assign a team')}
178178
</Text>
@@ -196,13 +196,13 @@ export function ScmProjectDetails({onComplete}: StepProps) {
196196
</Stack>
197197

198198
<Stack gap="sm">
199-
<Flex gap="md" align="center">
200-
<IconSiren legacySize="16px" />
199+
<Flex gap="md" align="center" justify="center">
200+
<IconSiren size="md" />
201201
<Text bold size="lg" density="comfortable">
202202
{t('Alert frequency')}
203203
</Text>
204204
</Flex>
205-
<Text variant="muted" size="lg" density="comfortable">
205+
<Text variant="muted" size="lg" density="comfortable" align="center">
206206
{t('Get notified when things go wrong')}
207207
</Text>
208208
<ScmAlertFrequency

0 commit comments

Comments
 (0)