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

Scroll speed is faster when FlatList is inverted #2500

Open
1 task done
BeeMargarida opened this issue Mar 21, 2023 · 1 comment
Open
1 task done

Scroll speed is faster when FlatList is inverted #2500

BeeMargarida opened this issue Mar 21, 2023 · 1 comment
Labels
bug: react-native Bug associated with upstream React Native vendor code bug

Comments

@BeeMargarida
Copy link

BeeMargarida commented Mar 21, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Describe the issue

When the inverted prop is true for the FlatList, the scroll speed is faster.
This can be checked in the Lists example: https://p9t5cp.sse.codesandbox.io/lists

It might be related to the logic in this.invertedWheelEventHandler, but looking at the code there's nothing that stands out as wrong.

Expected behavior

Scroll speed should remain the same if the list is inverted.

Steps to reproduce

  1. Go to https://p9t5cp.sse.codesandbox.io/lists
  2. Scroll with inverted disabled and note the scroll speed (with the trackpad or with the scroll wheel)
  3. Enable the inverted prop
  4. Check that the scroll is faster

Test case

https://p9t5cp.sse.codesandbox.io/lists

Additional comments

This issue was first spotted here.

@BeeMargarida BeeMargarida changed the title Scroll speed difference when lists are inverted Scroll speed is faster when FlatList is inverted Mar 21, 2023
@necolas
Copy link
Owner

necolas commented Mar 21, 2023

I think all issues related to Animated and FlatList should be directed towards this RN issue facebook/react-native#35263

Once RNWeb can import those modules from npm, they can be kept in sync across native and web, and any issues can be managed out of the RN repo only. This should make it a lot simpler going forward.

@necolas necolas added the bug: react-native Bug associated with upstream React Native vendor code label Sep 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug: react-native Bug associated with upstream React Native vendor code bug
Projects
None yet
Development

No branches or pull requests

2 participants