-
-
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
Docs: Add CSF 4 (experimental) snippets #30269
base: docs-csf-4
Are you sure you want to change the base?
Conversation
@@ -1,3 +1,93 @@ | |||
```js filename="Button.stories.js" renderer="common" language="js" tabTitle="CSF 3" |
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.
This diff is unfortunately hard to read. There are two required steps that aren't react
-specific:
common
snippets must be placed before thereact
ones- Add
tabTitle="CSF 3"
tocommon
snippets
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.
The tags-in-preview.md
diff is much easier to understand.
``` | ||
|
||
```js filename="Button.stories.js" renderer="svelte" language="js" tabTitle="CSF" | ||
import Button from './Button.svelte'; | ||
<!-- js & ts-4-9 (when applicable) still needed while providing both CSF 3 & 4 --> |
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.
For the time being, we will need react
versions of whatever languages are available for other renderers. In this case that means js
, ts
, and ts-4-9
.
I left this comment in there for an easy find/replace once we only have CSF 4 snippets.
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.
If you think this is too much of a burden, I can try to adjust the snippets logic to only require one language for the CSF 4 snippets. But I anticipate that being quite tricky, and I'd rather do the "dumb" solution.
d0371d2
to
6825384
Compare
View your CI Pipeline Execution ↗ for commit 619f909.
☁️ Nx Cloud last updated this comment at |
What I did
Add CSF 4 (experimental) versions of code snippets when
react
is the active renderer, using tabs:Checklist for Contributors
Testing
Manual testing
react
and non-react
renderers and for all languages (js
,ts
,ts-4-9
)Documentation
MIGRATION.MD
Checklist for Maintainers
When this PR is ready for testing, make sure to add
ci:normal
,ci:merged
orci:daily
GH label to it to run a specific set of sandboxes. The particular set of sandboxes can be found incode/lib/cli-storybook/src/sandbox-templates.ts
Make sure this PR contains one of the labels below:
Available labels
bug
: Internal changes that fixes incorrect behavior.maintenance
: User-facing maintenance tasks.dependencies
: Upgrading (sometimes downgrading) dependencies.build
: Internal-facing build tooling & test updates. Will not show up in release changelog.cleanup
: Minor cleanup style change. Will not show up in release changelog.documentation
: Documentation only changes. Will not show up in release changelog.feature request
: Introducing a new feature.BREAKING CHANGE
: Changes that break compatibility in some way with current major version.other
: Changes that don't fit in the above categories.