student-hub / acs-upb-mobile

A mobile application for students at ACS UPB.
MIT License
27 stars 22 forks source link

Improve loading UI #256

Open IoanaAlexandru opened 3 years ago

IoanaAlexandru commented 3 years ago

We're currently using centered CircularProgressIndicators whenever loading some data. For a nicer look, we could use a skeleton loader instead (e.g. skeleton_loader).

This isn't difficult to do so I'm marking it as a good first issue, as it's a good way to get familiar with building UI with flutter as well as the different components of the app. It will, however, take some time as we need to make sure this type of loading is consistently used throughout the entire app. The recommended workflow would be to create a skeleton_loading branch and then individual PRs (to be merged into skeleton_loading) for each separate component that is changed to use skeleton loading. This way, not all components need to be changed by the same person, and we can merge skeleton_loader into master once everything is done (similar to what we're currently doing with the web layout).