Overhaul on the typings of props, events and slots of all components from @svelteuidev/core. It uses svelte/elements to allow correct typings for each component.
⚠️ This is a big one, I tested it but it might introduce some bugs, we'll see.
Another changes:
Remove unused typings in internal (component props for errors)
IconRenderer size prop now supports SvelteUINumberSize type
Removed overrideInput prop from TextInput
Bumped svelte to 3.55 and svelte-check versions - locked @sveltejs/vite-plugin-svelte to <2 so that the storybook integration does not break, since storybook@7 is not ready yet
Moved typings in internal and styles to the types folder - still not sure if this is the best way of organizing stuff
Changed types files to .d.ts due to the way svelte-package generates types - it generates them in another folder inside package, which I can't change (for now we have a script that runs when we prepare for publishing that copies the types files into /types- investigating if there is a workaround to be made that does not require this). With .d.ts, those files are just copied to the correct location and give correct typings on watch (when debugging and developing).
Before submitting the PR, please make sure you do the following
Fixes #275.
Overhaul on the typings of props, events and slots of all components from
@svelteuidev/core
. It usessvelte/elements
to allow correct typings for each component. ⚠️ This is a big one, I tested it but it might introduce some bugs, we'll see.Another changes:
size
prop now supportsSvelteUINumberSize
typeoverrideInput
prop fromTextInput
svelte
to 3.55 andsvelte-check
versions - locked@sveltejs/vite-plugin-svelte
to <2 so that the storybook integration does not break, since storybook@7 is not ready yettypes
folder - still not sure if this is the best way of organizing stuff.d.ts
due to the waysvelte-package
generates types - it generates them in another folder insidepackage
, which I can't change (for now we have a script that runs when we prepare for publishing that copies the types files into /types- investigating if there is a workaround to be made that does not require this). With.d.ts
, those files are just copied to the correct location and give correct typings on watch (when debugging and developing).Before submitting the PR, please make sure you do the following
[@svelteui/core]
,[@svelteui/actions]
,[@svelteui/motion]
,[@svelteui/core]
,[core]
, or[docs]
.Tests
npm test
and lint the project withyarn lint
or just runyarn repo:prepush
and check to see if it's passing.