Hook to android keyboard events

[fakenickels]

I had some issues on Android and I could solve them with these fixes.

The issues were:

1. Keyboard flickering when tapping in one input (keyboard goes up) then tapping another input (keyboard goes down).
2. After solving that 1. happened that when tapping in one input then tapping another one the first one was getting focused again right after.

Tell if I need to do something before merging. And thanks for the awesome package!

[alvaromb]

This change isn't needed because you can pass keyboardShouldPersistTaps through KeyboardAwareScrollView props.

[fakenickels]

Yes, I just put it as default! But I can remove

[alvaromb]

Hello @grsabreu!

Thanks for this PR. I've added some feedback to your code. I will try to test and integrate this as soon as possible.

[fakenickels]

Thanks for the feedback! Sorry about the delay on replying

[alvaromb]

Can you rebase against master & squash your commits?

[fakenickels]

Ok! I'll do that ASAP

[fakenickels]

@alvaromb just finished what you proposed and pushed, tell me if anything else is needed
Thanks

[evollu]

@alvaromb any updates on this?

[alvaromb]

Can you change this to:

const isAndroid = Platform.OS === 'android'

It's not necessary to run the function every time you need to check if it's Android or not.

[alvaromb]

Hi @grsabreu @evollu!

Sorry for the delay. I would like @grsabreu to check the comments about the code I've just left. I would really like to merge this as soon as possible.

Thanks!

[sarovin]

The props contentInset work only on IOS

[sarovin]

You can use this:

<ScrollView //...props>
  {this.props.children}
  <View style={{ height: this.state.keyboardSpace }} />
</ScrollView>

https://github.com/sarovin/react-native-scrollview-smart/blob/master/lib/ScrollViewSmart.js

[fakenickels]

ok! I'll check the comments and update the PR ASAP

[fakenickels]

Closed in favor of #71