-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Addon-docs: Add Controls argument autodetection for svelte #12347
Conversation
Svelte docgen
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome -- pretty clean & works great! 💯 A few questions & requested changes.
examples/svelte-kitchen-sink/src/stories/addon-controls.stories.js
Outdated
Show resolved
Hide resolved
…s.js Co-authored-by: Michael Shilman <[email protected]>
my mistake, i commited a snapshot test versus an unit test. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing work @BlackFenix2! Bravo!!!! 🎉
Issue:
What I did
I have added autodetection capabilities for svelte components. i have create a custom webpack loader using sveltedoc-parser and populating the generated document into the ArgTypes.
How to test
run the svelte sample under examples/svelte-kitchen-sink and omit custom args