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

[Draft] Fix reverted virtualized list #2151

Closed
wants to merge 3 commits into from
Closed

[Draft] Fix reverted virtualized list #2151

wants to merge 3 commits into from

Conversation

azimgd
Copy link

@azimgd azimgd commented Oct 28, 2021

Expected Result:

  • When copying text in <FlatList inverted={true} /> that when you highlight lines of text, including across multiple messages, that the text is highlighted in order from top to bottom or bottom to top.
  • When pasting text that has been copied from Flatlist, that it shows up in the correct order.

Actual Result:

  • Copying is sporadic and 'jumps around' when you highlight text over multiple messages.
  • Pasting - When pasting copied text from multiple e.cash messages, it's out of order
Highlighting glitch Copy order

This has been reviewed and tested by multiple developers. Some references are:

Implementation

I have removed flatlist inversion (by inverted=true prop) which used [transform: translate] css hack and instead adding flatlist array items in reverse order [unshift instead of push]; It also reverses ScrollView event response received from browser [e.nativeEvent.configOffset *= -1];

Sidenote

I was not able to fix this issue by applying this code directly in React Native repo. The reason is that RN column-reverse won't anchor scroll position to the bottom on Mobile device as it does on Web
facebook/yoga#866 (comment)

@necolas
Copy link
Owner

necolas commented Mar 27, 2023

Closing this as list development is being consolidated in the React Native monorepo facebook/react-native#35263

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

Successfully merging this pull request may close these issues.

2 participants