-
Notifications
You must be signed in to change notification settings - Fork 377
chore(Button): Convert examples to typescript #6796
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
chore(Button): Convert examples to typescript #6796
Conversation
d04367a to
07fe9db
Compare
| isLoading={isPrimaryLoading} | ||
| variant="primary" | ||
| onClick={() => setIsPrimaryLoading(!isPrimaryLoading)} | ||
| {...(isPrimaryLoading && { spinnerAriaValueText: 'Loading' })} |
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 line is causing a problem when I hit the ts>>js button...
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 mechanism used to convert between TS and JS seems to break when spreading props (at least for Button). Can you point me to where we do the conversion so I can take a look?
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.
with about 70% confidence, I think that is what this line is supposed to do?
https://github.com/patternfly/patternfly-org/blob/main/packages/theme-patternfly-org/scripts/md/mdx-hast-to-jsx.js#L166
nicolethoen
left a 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.
Works great
issues with typescript -> javascript parsing will be resolved when this org PR merges. This PR does not need to be blocked.
|
Your changes have been released in:
Thanks for your contribution! 🎉 |
What: Closes #6793