react-simple-pull-to-refresh

react-simple-pull-to-refresh copied to clipboard

Corrected minor typos in the read me file.

jaedag

Keep FetchMore logic enabled when isPullable is disabled

1

The fetchMore logic doesn't work anymore if the isPullable flag is disabled. I don't think this is desired behavior. If you agree, then this would be a solid patch.

micheleppink

I found this this issue when i was working with dropdown in react , when i open the dropdown and select any option and move cursor any where down ,...

Abhii5496

When I "Pull to Refresh" and no data comes in, the animation continues to load, even after isPullable becomes false. How I tell the component to stop the loading animation...

CalvinJamesHeath

Don't work if `height: 100vh`

2

Don't work if i want on mobile do: ```css body, #app { height: 100vh; width: 100vw; } ```

beautyfree

your code ```js const getScrollToBottomValue = (): number => { if (!childrenRef || !childrenRef.current) return -1; const scrollTop = window.scrollY; // is the pixels hidden in top due to the...

Weibozzz

This is more of a feature request, but would love the ability to know if the user began the pull. I have a list of intractable items (both tap and...

helzapps

container onScroll listen

2

here is listening to the window scroll event, but what if the scroll area is a div not the window, it seems won't be able to work , especially `onFetchMore`...

javaxiu

Use optional chaining instead of non-null assert

2

Fix #81 Due to events like `onScroll` that this library listens to are not synchronous, they can be fired right after the component is already unmounted, causing `ref.current` to be...

johnsonsu

The following line is throwing an error due to `containerRef.current` being `null`. `react-simple-pull-to-refresh/build/index.esm.js` in onScroll at line 258:26 ```tsx /** * Check if user breached fetchMoreThreshold */ if (canFetchMore &&...

johnsonsu