Skip to content

fix(public-docsite-v9): render args table only for component stories#32467

Closed
dmytrokirpa wants to merge 1 commit into
microsoft:masterfrom
dmytrokirpa:fix/docsite-motion-pages
Closed

fix(public-docsite-v9): render args table only for component stories#32467
dmytrokirpa wants to merge 1 commit into
microsoft:masterfrom
dmytrokirpa:fix/docsite-motion-pages

Conversation

@dmytrokirpa

Copy link
Copy Markdown
Contributor

Previous Behavior

The Motion documentation is broken because the component is not specified in the Storybook's meta, and the ArgsTable is attempting to render the component's props table.

This is not a correct story definition from the Storybook's standpoint, as it doesn't satisfies SB's types.

image

New Behavior

As a temporary workaround, we could bypass ArgsTable rendering for stories without component specified. However, ideally, we should not create stories without components. Stories are meant for components, and if we need to document something else, we should create an mdx file with the documentation.

Related Issue(s)

@fabricteam

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

✅ No changes found

@dmytrokirpa

Copy link
Copy Markdown
Contributor Author

Closing as it duplicates #32465

@dmytrokirpa dmytrokirpa closed this Sep 5, 2024
@dmytrokirpa
dmytrokirpa deleted the fix/docsite-motion-pages branch September 5, 2024 10:36
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.

[Bug]: motion docs are broken

2 participants