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

fix: message view auto scrolls up when swiped up #5551

Open
preeesha opened this issue Feb 7, 2024 · 3 comments
Open

fix: message view auto scrolls up when swiped up #5551

preeesha opened this issue Feb 7, 2024 · 3 comments

Comments

@preeesha
Copy link

preeesha commented Feb 7, 2024

Description:

When the chat is at its bottom, if user swipes up a couple of times (3-4, sometimes 2 also), the message view scrolls up.

Environment Information:

  • Rocket.Chat Server Version: 6.6.0
  • Rocket.Chat App Version: 4.47.0.99999
  • Device Name: Realme 7
  • OS Version: Realme UI 3.0

Steps to reproduce:

  1. Open any chat view
  2. Swipe up a couple of times
  3. Observe chat going up

Expected behavior:

This is an unexpected behaviour. Chat must remain at the bottom.

Actual behavior:

Auto.Scroll.Up.mp4
@preeesha
Copy link
Author

preeesha commented Feb 7, 2024

@GleidsonDaniel, @diegolmello Please let me know your views on this one. I would love to work on this.

Thanks for your time and consideration
Prisha Gupta

@GleidsonDaniel
Copy link
Contributor

I managed to reproduce it, but I don't think it's something simple to solve hahaha.
However, nothing stops you from wanting to work on it 😅.

@preeesha preeesha closed this as completed Feb 9, 2024
@preeesha preeesha reopened this Feb 9, 2024
@diegolmello
Copy link
Member

Pretty sure it's related to FlatList inverted

const Container = ({ children }: { children: React.ReactElement }) =>

We've been patching it to fix inverted behavior (for both ScrollView and RefreshControl) while keeping it fast over the years, but right now we removed them completely and we're just applying Container above.

Since we're using stock FlatList now, removing Container should fix the issue, right? We need to test.
Does this happen with FlashList?

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

No branches or pull requests

3 participants