-
Notifications
You must be signed in to change notification settings - Fork 205
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
[Bug] Poor performance on mobile web #49
Comments
The same here |
Actually I think this is caused by a minor controller listener, which doesn't get disposed correctly. As I see you have many shimmer widgets in your list so I think this may be related to the problem I mentioned. See this pull request #51 @Andreigr0
|
Has anyone found a work around for this issue? |
I found the problem. It's caused by flutter's html renderer which is used by default on mobile browsers. Building with Unfortunately, this increases download size on mobile devices by about 2 MB, but this trade-off was barely noticeable in my tests and is worth the improved performance in my eyes (performance improves overall, but most noticeably for this plugin). @Andreigr0 @jayjahI tried your proposal as well before finding this, but (at least for me) it didn't change anything noticeable, I'm afraid. |
Screen record https://photos.app.goo.gl/y7fdonK3epG4swJp7
Steps to reproduce
example
(I just changed item count from 6 to 200 on Loading List screen) for web using either html or canvaskit:flutter build web --web-renderer html
orflutter build web --web-renderer canvaskit
This could require to use
example
as a package itselfbuild
folder:cd build/web
python -m http.server 8000
192.168.1.156:8000
Loading list
P.S. Maybe it's an issue of Flutter itself
The text was updated successfully, but these errors were encountered: