refactor(components): use explicit size tokens instead of spacing tokens

devin-ai-integration[bot] · cpurps22 · devin-ai-integration[bot] · commit 6b2a1b2695ff · 2025-09-09T20:50:32.000Z
- Replace var(--lp-spacing-400) var(--lp-spacing-500) with var(--lp-size-12) var(--lp-size-16)
- Update changeset description to reference size tokens for clarity
- Addresses PR feedback from matthewferry

Co-Authored-By: cpursley@launchdarkly.com &lt;cpursley@launchdarkly.com&gt;
diff --git a/.changeset/update-popover-padding.md b/.changeset/update-popover-padding.md
@@ -5,5 +5,5 @@
 feat(components): update DialogTrigger popover padding
 
 Updates DialogTrigger popover padding from uniform 8px to differentiated vertical/horizontal padding:
-- Vertical padding: 8px → 12px (var(--lp-spacing-400))
-- Horizontal padding: 8px → 16px (var(--lp-spacing-500))
+- Vertical padding: 8px → 12px (var(--lp-size-12))
+- Horizontal padding: 8px → 16px (var(--lp-size-16))
diff --git a/packages/components/src/styles/Popover.module.css b/packages/components/src/styles/Popover.module.css
@@ -37,7 +37,7 @@
 	}
 
 	&[data-trigger='DialogTrigger'] {
-		padding: var(--lp-spacing-400) var(--lp-spacing-500);
+		padding: var(--lp-size-12) var(--lp-size-16);
 	}
 
 	&[data-trigger='ComboBoxDialog'] {

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`&[data-trigger='DialogTrigger'] {`
`40`		`- padding: var(--lp-spacing-400) var(--lp-spacing-500);`
	`40`	`+ padding: var(--lp-size-12) var(--lp-size-16);`
`41`	`41`	`}`
`42`	`42`
`43`	`43`	`&[data-trigger='ComboBoxDialog'] {`