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
Fix for issue #1390 caused by secure component having multiple childs #1394
Conversation
…hilds Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
style={{ display: "none" }} | ||
ref={folderUpload} | ||
/> | ||
<> |
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.
Please use <Fragment>
for consistency
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 see most examples on react using this shorthand, we should start adopting it
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.
Then we would need to change across all the code. I think Fragment is better because is more clear what this is referring to.
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.
style={{ display: "none" }} | ||
ref={folderUpload} | ||
/> | ||
<> |
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.
Then we would need to change across all the code. I think Fragment is better because is more clear what this is referring to.
Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
style={{ display: "none" }} | ||
ref={folderUpload} | ||
/> | ||
<Fragment> |
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.
Thank you! =D
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.
Learning from it:
A fragment lets us group multiple JSX elements without adding an extra wrapper node.
Fixes #1390
Signed-off-by: Daniel Valdivia 18384552+dvaldivia@users.noreply.github.com