Skip to content
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 Methods to web components ArgsTable #12413

Merged
merged 1 commit into from Sep 8, 2020
Merged

Addon-docs: Add Methods to web components ArgsTable #12413

merged 1 commit into from Sep 8, 2020

Conversation

Shackless
Copy link
Contributor

@Shackless Shackless commented Sep 8, 2020

Issue: #12412

What I did

Some web component frameworks like Stencil are able to generate documentation for public methods of web components.
These should be shown in the ArgsTable of Storybook next to Attributes, Events, Slots etc. but are currently missing.

I added 'methods' to the custom elements interface and extractor and also included a demo entry to the custom-elements.json file in the web components kitchen-sink to display the results.

How to test

  • run web components kitchen-sink and find the new Methods section in the ArgsTable

Screenshot

image

@shilman shilman changed the title docs: ArgsTable is missing Methods section for web components Addon-docs: Add Methods section for web components ArgsTable Sep 8, 2020
Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for putting this together! 🎉

@shilman shilman changed the title Addon-docs: Add Methods section for web components ArgsTable Addon-docs: Add Methods section to web components ArgsTable Sep 8, 2020
@shilman shilman changed the title Addon-docs: Add Methods section to web components ArgsTable Addon-docs: Add Methods to web components ArgsTable Sep 8, 2020
@shilman shilman merged commit 6d5faca into storybookjs:next Sep 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants