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

verticalOffset prop #2

Merged
merged 5 commits into from
Dec 14, 2018
Merged

verticalOffset prop #2

merged 5 commits into from
Dec 14, 2018

Conversation

AlexisJamin
Copy link
Contributor

Adds verticalOffset prop to fix bug with react-navigation header.

Adds verticalOffset prop to fix bug with react-navigation header.
src/index.js Outdated
let { children, backgroundColor } = this.props;
let { children, backgroundColor, verticalOffset } = this.props;

//adds verticalOffset prop to bottom
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you remove the comment? coz the name of the variable already explained its usage.

@just4fun
Copy link
Owner

Could you please also update the doc?

Copy link
Contributor Author

@AlexisJamin AlexisJamin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deleted comment in file.

src/index.js Outdated Show resolved Hide resolved
Co-Authored-By: AlexisJamin <[email protected]>
Copy link
Contributor Author

@AlexisJamin AlexisJamin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added @JelteF work.

@just4fun just4fun mentioned this pull request Dec 13, 2018
src/index.js Outdated Show resolved Hide resolved
@just4fun
Copy link
Owner

I'd like to accept this PR to resolve the integration issue #1 once the #3 has been included into this PR and all the comments got resolved.

Copy link
Contributor Author

@AlexisJamin AlexisJamin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

!== ok

Updated doc with verticalOffset prop.
Copy link
Contributor Author

@AlexisJamin AlexisJamin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated Readme with verticalOffset prop.

@just4fun just4fun merged commit 546c7f9 into just4fun:master Dec 14, 2018
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