-
Notifications
You must be signed in to change notification settings - Fork 1
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/hooks - eslint & tests #5
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.
Minor questions!
'rules': { | ||
'indent': [ | ||
'error', | ||
4 |
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 prefer 2! But let it be ;-)
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 prefer 4 :) but we can agree on any setting!
|
||
class TopBar extends Component { | ||
class TopBar 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.
Why?
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.
@Tuhaj it does not matter - both effect is the same (i.e. either importing React, {Component}
or just React
). The only reason why I picked React.Component
is that Component
is a generic name that any library may export. By using React.Component
we strictly say "hey, this will be React.Component!" - as Component
is a common name and may be used by other libraries.
@@ -22,16 +23,20 @@ class TopBar extends Component { | |||
<Text type="basic" color={ Colors.white } display={ 'inline' }> | |||
Zaloguj się przez GitHub | |||
</Text> | |||
<Button marginLeft='10px' onClick={this.onClick} | |||
type="primary"> | |||
<Button marginLeft='10px' onClick={this.onClick} type="primary"> |
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.
here primary
in double quotes?
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.
To be honest I haven't even noticed, as my changes only re-formatted this part. Will check this within next PR. Good catch!
fixes #6