Update after PR comments

Shopify · Mar 11, 2022 · 35d6641 · 35d6641
1 parent 099486f
commit 35d6641
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 20 deletions.
diff --git a/documentation/docs/index.md b/documentation/docs/index.md
@@ -17,7 +17,7 @@ Check out [the Vault project](https://vault.shopify.io/projects/22845) to learn
 
 ### Adding a package to your project
 
-Add the package to your project via `yarn add @shopify/flash-list recyclerlistview@3.1.0-beta.5` and run `pod install` in the `ios` directory.
+Add the package to your project via `yarn add @shopify/flash-list recyclerlistview@3.1.0-beta.6` and run `pod install` in the `ios` directory.
 
 :::note
 [`recyclerlistview`](https://github.com/Flipkart/recyclerlistview) is a peer dependency of `@shopify/flash-list`.

diff --git a/documentation/docs/known-issues.md b/documentation/docs/known-issues.md
@@ -23,26 +23,22 @@ This is something that hasn't been implemented yet. It will be part of the first
 
 It's not implemented yet but will be part of final release.
 
-### 4) `onEndReachedThreshold` value isn't working
-
-Please note that value accepted is in dp/px and not a multiplier to window size. Please make the change accordingly. We will address this in the future.
-
-### 5) `onEndReached` event doesn't have `distanceFromEnd`
+### 4) `onEndReached` event doesn't have `distanceFromEnd`
 
 This value is reported as 0. We don't have plans to implement this right now. Please provide feedback if this is important to you.
 
-### 6)`scrollToIndex` doesn't have `offset` support
+### 5)`scrollToIndex` doesn't have `offset` support
 
 The list only guarantees to bring the given index in view port (as close to the top as possible). There is no support for additional customization.
 
-### 7) `renderItem` callback doesn't have all features
+### 6) `renderItem` callback doesn't have all features
 
 As of now we only provide relevant data and index. No plans to change this.
 
-### 8) `viewability` related callbacks are missing
+### 7) `viewability` related callbacks are missing
 
 We are planning to add a version of this but we can't guarantee if it will be exactly same a `FlatList`.
 
-### 9) Horizontal lists do not change height to match tallest child
+### 8) Horizontal lists do not change height to match tallest child
 
 As of now, horizontal lists need a fixed height. It's not possible to resize it based on height of the tallest child item. This is because children have absolute positions and cannot expand the parent. This is something we will address in the future. Please provide feedback if this is important to you. Using `onLayout` to measure and resize is a possible workaround but it can cause visible movement on screen.
diff --git a/fixture/package.json b/fixture/package.json
@@ -19,7 +19,7 @@
     "react-native-gesture-handler": "^2.3.1",
     "react-native-safe-area-context": "^3.3.2",
     "react-native-screens": "^3.13.1",
-    "recyclerlistview": "3.1.0-beta.5"
+    "recyclerlistview": "3.1.0-beta.6"
   },
   "devDependencies": {
     "babel-plugin-module-resolver": "^4.1.0",

diff --git a/fixture/src/Twitter.tsx b/fixture/src/Twitter.tsx
@@ -22,10 +22,6 @@ const Twitter = () => {
         estimatedItemSize={150}
         ItemSeparatorComponent={Divider}
         data={tweets}
-        onEndReachedThreshold={0.5}
-        onEndReached={() => {
-          console.log("onEndReached");
-        }}
       />
     </RecyclerFlatListPerformanceView>
   );

diff --git a/fixture/yarn.lock b/fixture/yarn.lock
@@ -4041,10 +4041,10 @@ recast@^0.20.3:
     source-map "~0.6.1"
     tslib "^2.0.1"
 
-recyclerlistview@3.1.0-beta.5:
-  version "3.1.0-beta.5"
-  resolved "https://registry.yarnpkg.com/recyclerlistview/-/recyclerlistview-3.1.0-beta.5.tgz#ef379cc51f4e73d1ccae68b5b95f459b3d665a98"
-  integrity sha512-9s75KHbOxuSejAS76Th8tSW1youTHQXnJR+oI1rYoesaHqie3JdvQZww6akinydSAJ4s0SeMto2pTq1IiapYSQ==
+recyclerlistview@3.1.0-beta.6:
+  version "3.1.0-beta.6"
+  resolved "https://registry.yarnpkg.com/recyclerlistview/-/recyclerlistview-3.1.0-beta.6.tgz#b92d2daae6d871efe7f500da1064a58175832516"
+  integrity sha512-qfhvhgDQYfQHC5FhNsiztpF7yeI7tD3Dj58xrlSOXrDSoUJVvctrtX2hmV0ytLkPdHCnGkSpr9HbeRK/rR29Gw==
   dependencies:
     lodash.debounce "4.0.8"
     prop-types "15.5.8"

diff --git a/package.json b/package.json
@@ -26,7 +26,7 @@
     "@babel/runtime": "*",
     "react": "*",
     "react-native": "*",
-    "recyclerlistview": "3.1.0-beta.5"
+    "recyclerlistview": "3.1.0-beta.6"
   },
   "devDependencies": {
     "@babel/core": "^7.17.5",