Skip to content

Commit 0ebcdfe

Browse files
Merge remote-tracking branch 'origin/master' into cy/remove_ta_be
2 parents eb09ec7 + 045eb21 commit 0ebcdfe

File tree

1,892 files changed

+73078
-30650
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

1,892 files changed

+73078
-30650
lines changed

.agents/skills/cell-architecture/SKILL.md

Lines changed: 268 additions & 0 deletions
Large diffs are not rendered by default.

.agents/skills/design-system/SKILL.md

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ import {Container} from '@sentry/scraps/layout';
5151

5252
// ❌ Don't create styled components
5353
const Component = styled('div')`
54-
padding: ${space(2)};
54+
padding: ${p => p.theme.space.md};
5555
border: 1px solid ${p => p.theme.tokens.border.primary};
5656
`;
5757

@@ -119,7 +119,7 @@ import {Grid} from '@sentry/scraps/layout';
119119
const Component = styled('div')`
120120
display: grid;
121121
grid-template-columns: repeat(3, 1fr);
122-
gap: ${space(2)};
122+
gap: ${p => p.theme.space.md};
123123
`;
124124

125125
// ✅ Use Grid primitive
@@ -147,7 +147,7 @@ import {Stack} from '@sentry/scraps/layout';
147147
const Component = styled('div')`
148148
display: flex;
149149
flex-direction: column;
150-
gap: ${space(2)};
150+
gap: ${p => p.theme.space.md};
151151
`;
152152

153153
// ✅ Use Stack primitive (automatically column direction)
@@ -204,7 +204,7 @@ import {Text} from '@sentry/scraps/text';
204204
// ❌ Don't create styled text components
205205
const Label = styled('span')`
206206
color: ${p => p.theme.tokens.content.secondary};
207-
font-size: ${p => p.theme.fontSizes.small};
207+
font-size: ${p => p.theme.font.size.sm};
208208
`;
209209

210210
// ❌ Don't use raw elements
@@ -242,7 +242,7 @@ import {Heading} from '@sentry/scraps/text';
242242

243243
// ❌ Don't style heading elements
244244
const Title = styled('h2')`
245-
font-size: ${p => p.theme.fontSize.md};
245+
font-size: ${p => p.theme.font.size.md};
246246
font-weight: bold;
247247
`;
248248

@@ -401,7 +401,7 @@ Container supports `margin` props but they are deprecated. Use `gap` on parent c
401401
```tsx
402402
// ❌ Don't use margin between children
403403
const Child = styled('div')`
404-
margin-right: ${p => p.theme.spacing.lg};
404+
margin-right: ${p => p.theme.space.lg};
405405
`;
406406

407407
// ✅ Use gap on parent container
@@ -421,7 +421,7 @@ const Component = styled('div')`
421421
display: flex;
422422
flex-direction: column;
423423
color: ${p => p.theme.tokens.content.secondary};
424-
font-size: ${p => p.theme.fontSize.lg};
424+
font-size: ${p => p.theme.font.size.lg};
425425
`;
426426

427427
// ✅ Split into layout and typography primitives

0 commit comments

Comments
 (0)