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

android app works slow after using this library #51

Open
shivam0831 opened this issue Jun 1, 2021 · 4 comments
Open

android app works slow after using this library #51

shivam0831 opened this issue Jun 1, 2021 · 4 comments

Comments

@shivam0831
Copy link

shivam0831 commented Jun 1, 2021

while using this library in android after using this library app works very slow. any suggestions to solve this?

@shivam0831 shivam0831 changed the title android app becomes slow android app works slow after using this library Jun 1, 2021
@VincentCATILLON
Copy link
Owner

Hi @shivam0831 , I can't give your tips without any code or informations about your usage of the lib.

As it renders all particles at the same time in parallel, you can obviously use a reasonable amount of items, like 100 or 150 max, to avoid performances issues.

I'd like to add that this lib uses native rendering and RN API (Animated / Animations), to animate them, so I don't see how I can improve it :)

@apoorv-kansal-trysolvio

@shivam0831 I found that you can conditionally render the Confetti component when you want to show confetti. Maybe this part can be put in documentation as a warning. Putting 200 (as shown in the example) is quite a lot and causes app to load.

Also, @VincentCATILLON maybe if autoStart is set to start to false you can not create the items in the starting. And only when start method is called?

@echolove38

This comment was marked as abuse.

@alz10
Copy link

alz10 commented Feb 22, 2022

@VincentCATILLON Any update on the library? There are Pull Requests that are very useful waiting for you to merge. I'm also experiencing this slow performance on android. I had to lower the confetti count to 100 instead of 200. The performance delay is still a bit noticeable tho

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

No branches or pull requests

5 participants