-
Notifications
You must be signed in to change notification settings - Fork 101
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(ui-radio-input,ui-source-code-editor): add functional examples t… #1842
base: master
Are you sure you want to change the base?
Conversation
…o SourceCodeEditor and class-based examples to RadioInputGroup Closes: INSTUI-4419
|
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.
Good work
``` | ||
|
||
- ```js | ||
const languages = { |
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 source code has extra padding in the functional examples
const handleChange = function (event, value) { | ||
console.log(value) | ||
- ```js | ||
class Example extends React.Component { |
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.
in the (far away) future we will probably only need functional examples for components so I'd suggest not to add class based ones if we don't have them already
…o SourceCodeEditor and class-based examples to RadioInputGroup
Closes: INSTUI-4419
TEST PLAN:
- test all the class-based examples in RadioInputGroup if they work the same as the functional examples and without error
- test all the functional examples in SourceCodeEditor if they work the same as the class-based and without error