Adding onViewableItemsChanged to SectionList changes the items passed to each section's keyExtractor
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 127k
- Forks
- 25.3k
- Avg merge
- 1d 23h
- Merged PRs (30d)
- 4
Description
Description
We wrote a SectionList component where each section defines it's own keyExtractor. One of the sections uses a key from a nested object in each item in the section. It broke when we wanted to add visibility tracking to the section list, and it seems like the section objects are being run through the keyExtractor but only when onViewableItemsChanged prop is provided.
I expect that only items within each section should be run through the section's keyExtractor function, and that the behavior is consistent regardless of what props are provided.
Steps to reproduce
- Open expo snack linked below
- Run it on android or iOS device and observe it works
- Uncomment the
onViewableItemsChangedline from the SectionList and observe the break
React Native Version
0.75.3
Affected Platforms
Runtime - Android, Runtime - iOS
Output of npx react-native info
See expo snack for reproduction
Stacktrace or Logs
Cannot read property 'id' of undefined
Reproducer
https://snack.expo.dev/V1Ad0VTld860Zdt8n-qb7
Screenshots and Videos
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the linked Expo Snack and reproduce the failure on Android or iOS by toggling the SectionList onViewableItemsChanged prop. Trace how visibility tracking invokes the section keyExtractor and verify that only items within each section are passed to it consistently with or without that prop.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react-native
- Domain
- mobile
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100