Skip to content
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

Using fragment instead of the div tag #1676

Closed
wants to merge 1 commit into from

Conversation

ali-master
Copy link

Hi there,

Since we are using the latest version of React.js then we can use the Fragment feature from now on. And I think it will be helped other users for using this awesome feature in their codes.

@jorrit
Copy link

jorrit commented Oct 2, 2018

Have you considered the shorthand <> syntax for fragments?

@jorrit
Copy link

jorrit commented Oct 2, 2018

Also, can you update the unit test as well?

@ali-master
Copy link
Author

First question:
No, but I can update it asap.

Second question:
Yes I will update unit tests

@piglovesyou
Copy link
Collaborator

This issue is one of the reasons working on #1797, thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants