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

Upgrade dependencies #128

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

Conversation

maznnwell
Copy link

I've upgraded the flutter_staggered_grid_view to 0.7.0 and replace the defunct SliverStaggeredGrid with StaggeredGrid.

I've delete one unused dependency and upgraded others to their latest versions. Some changes to the code where needed to handle the new APIs.

Now the app compiles with Flutter 3.24.3. But this caused a rendering bug: leanflutter/window_manager#464

I don't know exactly what caused it. I was able to fix it by adding a progress indicator while the lottie animation from the splash screen was being loaded.

I also fix a bug that was preventing my from compiling in release mode.

@KingGorrin
Copy link
Collaborator

That was easier than I thought. Very nicely done.

I was able to build on Windows using flutter 3.24.3. I did some quick testing and everything seems to work, maybe even slightly faster than before. I will do some more extensive testing on Linux and Windows later this week.

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

Successfully merging this pull request may close these issues.

2 participants