robcalcroft / react-native-in-app-notification

:bell: Customisable in-app notification component for React Native
MIT License
269 stars 78 forks source link

feat: Allow setting top offset value #8

Closed luizeduard0 closed 6 years ago

luizeduard0 commented 6 years ago

This was useful in my scenario. I had a fixed Header and a scroll view beneath it. The Notification was showing correctly, but when swiping it down it was possible to see the empty notification container. Setting a top offset resolves the issue. I believe the best practice would be to hide/show notification instead of just animating it up.

luizeduard0 commented 6 years ago

@robcalcroft there you go! (:

craig8two commented 6 years ago

How do I use this feature? Thanks for the great work.

robcalcroft commented 6 years ago

Whoops I think I forgot to upload to npm. I’ll make a note to do it this eve when I get home 😊

craig8two commented 6 years ago

Thank you.

robcalcroft commented 6 years ago

@craig8two 2.2.0 is up 😄

craig8two commented 6 years ago

@robcalcroft thanks, I'll give it a go soon...