-
Notifications
You must be signed in to change notification settings - Fork 18
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
Update NavBarContainer.js #34
Update NavBarContainer.js #34
Conversation
Added support for hiding navbar per route Style improvements
is the extra indentation on the if statements on purpose? |
Nope. let me fix that. |
fixed indent
I wish atom had a better way of switching indentation methods per file. Currently it's global, and i use tabs. I try to remedy all these misindentations manually, but this slipped through. sry :( |
if (this.props.currentRoute.trans) { | ||
trans = {backgroundColor: 'transparent'}; | ||
} | ||
else { |
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.
This should probably be consistent with the One True Brace in the rest of the code.
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.
indeed.
Mikael, this has been fixed. Let me know if something else pops up. |
Update NavBarContainer.js
Added support for hiding navbar per route
Style improvements