nimblehq / nimble-crypto-ios

This is an sample Crypto market prices ap built with SwiftUI and modularization architecture. The application data is making use of the free APIs provided by Coingecko.
MIT License
33 stars 8 forks source link

[#62] [Integrate] As a user, I can pull to refresh data on the Home screen #88

Closed Thieurom closed 1 year ago

Thieurom commented 1 year ago

What happened 👀

Insight 📝

At first, I attempted to use the .listStyle(.grouped) for the List, using the built-in Section (with header). However, I faced many issues with styling (background color, padding, etc, ...) the section, rows, and the List itself. Even the workaround like UITableView.appearance(), UICollectionView.appearance() doesn't work anymore. So I just stick with the `.listStyle(.plain) and custom Section and Header.

Proof Of Work 📹

https://user-images.githubusercontent.com/17972674/211231996-09c44360-f70c-43e1-90ff-4fd3d60a5f4e.mov

A case where the app lost the internet connection when initially launches

https://user-images.githubusercontent.com/17972674/211244054-81d5e49f-064c-498f-8c8b-3d566645e88b.mov

Thieurom commented 1 year ago

As a side note, can you also have a record case that first opening the application with no internet, then trigger a reload with internet connection to showcase the new renderIf call for the P.o.W? That would be a great demonstration 🙏

A PoW video for the case of launching the app without an internet connection was added 🙏