-
Notifications
You must be signed in to change notification settings - Fork 58
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
feat: upstream FormikField #1054
Conversation
…elds and Formik easier to work with.
Demo starting at https://react-components-1054.demos.haus |
@@ -0,0 +1,75 @@ | |||
import React from "react"; |
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.
I think the docs doesn't render as expected. For instance:
- when pressing
Show code
, the entire code is presented, but I assume we only want the bit fromrender
to be displayed. - the default value of component prop in props table is not listed in DEFAULT column.
The first issue also appears in Card
stories, which is one of the few other .stories.tsx
docs. There are also some slight inconsistencies in presentation between .stories.tsx
and .stories.mdx
docs e.g. the Props header missing in .stories.tsx
docs.
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.
I think the docs doesn't render as expected. For instance:
- when pressing
Show code
, the entire code is presented, but I assume we only want the bit fromrender
to be displayed.
There's an open issue here, with a workaround that I've applied to both files: storybookjs/storybook#22281.
- the default value of component prop in props table is not listed in DEFAULT column.
It looks like the table couldn't handle the Input
component as a default but this can be set via a jsdoc string so I've done that.
The first issue also appears in
Card
stories, which is one of the few other.stories.tsx
docs. There are also some slight inconsistencies in presentation between.stories.tsx
and.stories.mdx
docs e.g. the Props header missing in.stories.tsx
docs.
It looks like it was decided not to customise the page too much and stick with the CSF defaults: #1010 (comment).
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.
Thanks for the explanation and links, and for also fixing the inconsistency in Card
stories!
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.
LGTM 👍
🎉 This PR is included in version 0.51.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Done
QA
Storybook
To see rendered examples of all react-components, run:
QA in your project
from
react-components
run:Install the resulting tarball in your project with:
QA steps
Fixes
https://warthogs.atlassian.net/browse/WD-9813
Fixes: #957.