TheWidlarzGroup / rn-emoji-keyboard

Super performant, lightweight, fully customizable emoji picker 🚀
https://thewidlarzgroup.github.io/rn-emoji-keyboard/
MIT License
326 stars 59 forks source link

Emoji disappears when moving between navigator screens #113

Closed denissdubinin closed 1 year ago

denissdubinin commented 1 year ago

Describe the bug Emoji disappears when moving between navigator screens by dragging from the left. It is happening only after the component is scrolled, even when you scroll bar to Y=0 position. It stays invisible unless you scroll again - touching is not working, only scrolling. The issue happens on both Xcode emulator and real device.

To Reproduce Steps to reproduce the behavior:

  1. Put emoji component into navigator screen
  2. Navigate to your screen from other screen.
  3. Scroll the emoji component
  4. Try going to the previous screen by dragging it from the left

Expected behavior Emoji component does not disappear and is rendered only when you stay on same screen and scroll it.

Screenshots https://user-images.githubusercontent.com/15890438/210950353-7f938bfa-65b7-4afe-839b-5c8940661cfa.MP4

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context

"@react-navigation/bottom-tabs": "^6.5.0",
"@react-navigation/native": "^6.1.0",
"@react-navigation/native-stack": "^6.9.5",
"react": "18.1.0",
"react-native": "0.70.6",
"react-native-safe-area-context": "^4.4.1",
"react-native-screens": "^3.18.2",
"rn-emoji-keyboard": "^1.1.0",
"yarn": "^1.22.19"
mateki0 commented 1 year ago

Hi, thanks for reporting this bug. I created a PR with fix. However I've discovered that it only appears on screen from @react-navigation/native-stack and the ones from @react-navigation/native works without any problem. The fix and new version should be deployed in this week but if you want instant fix and it's possible in your application you can consider changing the navigation library :)

denissdubinin commented 1 year ago

Thank you!!! I have tested and the screen is working properly now. Let me know if I can close the task or we wait for merge/release

mateki0 commented 1 year ago

You're welcome, I'm happy that I could help you :) The issue will close automatically after merging PR