search

svelteuidev / svelteui

SvelteUI Monorepo
https://svelteui.dev
MIT License
1.28k stars 64 forks source link

Default theme spacing doesn't work #335

Closed notramo closed 1 year ago

notramo commented 1 year ago

When using override={{ padding: '$md' }}, the element has 0 padding. Could it be the reason that the default theme lacks the px unit after the numbers?

BeeMargarida commented 1 year ago

We have $mdPX that adds the px after. Not a clean solution, but it's what we are using when necessary

notramo commented 1 year ago

Why is the px left out from the default $md size?

BeeMargarida commented 1 year ago

We sometimes need to do operations on the values, like theme.space.xs / 2, so we went with this approach