-
-
Notifications
You must be signed in to change notification settings - Fork 44
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/styled component #114
Feat/styled component #114
Conversation
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.
Looks good to me
Left few suggestions
also, what do you think of making hello.js
-> hello.jsx
?
You can convert that to class component too if you want!
You can now make changes in your local machine and then push it to your branch itself. Thos will be reflected here automatically |
Actually there's no hello.js in this commit I move it into a hello folder and put index.js and style.js inside. I guess that's better and it will make our import statement shorter too. But if we want to keep it as hello.jsx, we can create a folder called styled with same level with test, This is the current structured: This is the structured I suggest if we want to keep file name as hello.jsx: What do you think @anikethsaha ? |
Ohh sorry, my bad...I meant |
Yea I faced that too. Actually I changed all components to |
Awesome then! Please check it once more :) Thank you |
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 🎉
@all-contributors please add @wincentrtz for code |
I've put up a pull request to add @wincentrtz! 🎉 |
This is what i have done: