-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[Help needed] Vote for React Native's feature requests so we can improve the plugin #203
Comments
I am doing it now. |
Done! I added a pull request to add visibility to this issue on readme file. |
Done! |
1 similar comment
Done! |
+! |
Done, thanks for your efforts on this! |
Dear @bd-arc , I think the snapToInterval for Android has been merge to RN master last time I check. facebook/react-native@b2848a5 |
Hey @ssssssssssss, Thanks for keeping me posted. Unfortunately, I don't think that the feature has been implemented yet since two PRs address it but none has been merged so far... |
@ssssssssssss Apparently you were right, but the feature was added in another commit: facebook/react-native@ddd65f1 Can't wait to see version |
That's awesome! Hopefully a vertical snap PR will be accepted soon. On the two PRs mentioned by @bd-arc is deemed as not as useful, but now that we have the horizontal one merged it's a good time to do the vertical too. |
@Adirael I agree with you, and that's exactly what I said in this comment to one of the PR. Unfortunately, if |
+1 |
This recent commit smells pretty good! The version |
@bd-arc I linked it in the community discussion to try and get it in 0.57.4 |
@kelset gave me that link : facebook/react-native@ef7e99c Turns out it's already here in 0.57 @bd-arc , so what are the next steps |
@Titozzz Well, a huge bunch of tests awaits us then! We need to see how props If someone wants to jump in and create a few public test cases on Snack, this would be awesome! There is one problem though: we still don't have a proper callback mechanism. The aim is to ditch the current hacky solution altogether, but there is no way to do that until something is implemented on the native side by the Facebook team... |
Update? for this
|
Take a look at https://github.com/CrowdLinker/react-native-pager Just as an inspiration for future development. Still love this library tho |
Looks good. Don't appreciate the offtopic spam though. |
@RWOverdijk I appreciate the intent ;-) A lot of goodness might come from what @tobiaslins has created though. @tobiaslins Well, I find your plugin really interesting! Just took a look at the source code and it seems you're on your way of achieving what I've been postponing for years — see #40 and #250 if you want to know what I mean. Quick questions:
|
@bd-arc The creator is @ajsmth maybe he can answer you this questions. Would love to see you working out something together!! Keep up the great work! |
hey there -- i would say that the repo is still immature so please take my answers with a grain of salt
cheers |
@ajsmth Thanks a lot for your answers! I'm really interested in what you've created because using such a base for this plugin would mean no more pesky As I see it, the tricky parts are:
You seem to have a great vision about dealing with all this. If you are, or anyone else is, interested in adding those features to your pager, I'll be more than happy to "connect the dots" with the rest and make the end result the new carousel! Let me know what you think. |
no problem! from what i can tell the only missing feature from your list there is the loop feature. as i said above, i've been focused on a few other things at the moment so this will likely take some time, but it's something i'll keep in mind. if anyone else is willing to give this a shot, feel free to reach out |
I have a suggestion. While scrolling from one card to another in First example gif we need to add a scroll animation too. i.e) If we scroll a nook means it shows a scroll animation. I need that support too in my app. So we can show the scroll animation here |
It was more than time to give this plugin a heavy lifting! Happy to report that 90% of the job is already done. Now I need you all to test, provide feedback, and even jump in if you feel like it 😉 |
This comment was marked as spam.
This comment was marked as spam.
Discord spoke about their issues with FlatList in this blog post: https://discord.com/blog/how-discord-achieves-native-ios-performance-with-react-native Their solution was a new, custom-made FastList component: https://gist.github.com/derekstavis/d8f6a00e14a4259224aafafab14ffd3e Perhaps this can help you with performance and the other issues you've listed. |
Hey there,
We're really pleased with how the plugin has grown over the past few months: there is now a large user base and it is used in numerous apps. We're proud of that and we want to thank you all for your appreciation and for making the most out of it! 😊 You've motivated us to devote countless hours maintaining and regularly improving the plugin, and we've been happy to give back to the community.
But now we're starting to feel tired of fighting React Native's core again and again, to work around the many issues of the
FlatList
andScrollView
components while knowing that the next critical one is just around the corner.Now is the time to let the React Native team know how badly we need them to improve those components! 💢 (and then we'll make the plugin even better, of course 😉)
The main issues of the
ScrollView/VirtualizedList/FlatList
components are the following:scrollEnd
eventscrollTo
method doesn't accept any callbackscrollTo
animation is quite brutalReact Native has a dedicated canny for feature requests. Here are the ones that need your vote the most:
If you find other feature requests or React Native issues that are relevant, do not hesitate to list them in this thread so we can vote together as a group and help putting the focus on those.
Remember that every vote counts ✊
The text was updated successfully, but these errors were encountered: